<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:s_j_newbury@yahoo.co.uk" title="Steven Newbury <s_j_newbury@yahoo.co.uk>"> <span class="fn">Steven Newbury</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - fix build with gcc link time optimizer"
href="https://bugs.freedesktop.org/show_bug.cgi?id=83669">bug 83669</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>s_j_newbury@yahoo.co.uk
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - fix build with gcc link time optimizer"
href="https://bugs.freedesktop.org/show_bug.cgi?id=83669#c12">Comment # 12</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - fix build with gcc link time optimizer"
href="https://bugs.freedesktop.org/show_bug.cgi?id=83669">bug 83669</a>
from <span class="vcard"><a class="email" href="mailto:s_j_newbury@yahoo.co.uk" title="Steven Newbury <s_j_newbury@yahoo.co.uk>"> <span class="fn">Steven Newbury</span></a>
</span></b>
<pre>(In reply to Marc Dietrich from <a href="show_bug.cgi?id=83669#c11">comment #11</a>)
<span class="quote">> sorry for spamming this bug, here is a better patch to fix the problem
>
> diff --git a/src/mesa/Makefile.am b/src/mesa/Makefile.am
> index e71bccb..bbdb36f 100644
> --- a/src/mesa/Makefile.am
> +++ b/src/mesa/Makefile.am
> @@ -152,7 +152,6 @@ libmesagallium_la_LIBADD = \
>
> libmesa_sse41_la_SOURCES = \
> main/streaming-load-memcpy.c
> -libmesa_sse41_la_CFLAGS = $(AM_CFLAGS) -msse4.1
>
> pkgconfigdir = $(libdir)/pkgconfig
> pkgconfig_DATA = gl.pc
> diff --git a/src/mesa/main/streaming-load-memcpy.c
> b/src/mesa/main/streaming-load-memcpy.c
> index 8427149..94b0e0a 100644
> --- a/src/mesa/main/streaming-load-memcpy.c
> +++ b/src/mesa/main/streaming-load-memcpy.c
> @@ -26,7 +26,7 @@
> *
> */
>
> -#ifdef __SSE4_1__
> +#ifdef USE_SSE41
> #include "main/macros.h"
> #include "main/streaming-load-memcpy.h"
> #include <smmintrin.h>
> @@ -34,7 +34,7 @@
> /* Copies memory from src to dst, using SSE 4.1's MOVNTDQA to get streaming
> * read performance from uncached memory.
> */
> -void
> +void __attribute__ ((target("sse4.1")))
> _mesa_streaming_load_memcpy(void *restrict dst, void *restrict src, size_t
> len)
> {
> char *restrict d = dst;</span >
Maybe you have, but can you ensure USE_SSE41 is only defined when __SSE4_1__ is
also defined. There's a common misconception that all ISA extensions are
always available with recent gcc releases, but this is only the case with the
popular binary distribution generic compiler packages. If gcc is built for a
specific CPU target it does not support code generation for other CPUs even if
the assembler does.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>