On Sat, Nov 17, 2007 at 12:14:46PM +0100, Marc Olzheim wrote:
> For the MACHINE_ARCH == "amd64" & ${CC} != "icc" case,
> the -mno-sse3 flag that *is* set for MACHINE_ARCH == "i386", is not set.
> 
> I don't think the current gcc actually uses sse3 anywhere in the kernel
> compile, as my kernels are working as normal, but for the sake of
> completeness, I suggest the attached patch. (For CURRENT as well as
> RELENG_7)

It's /usr/src/sys/conf/kern.mk:1.46 that misses the fact that it can be
set for amd64 as well, not just for i386.

Should I file a PR to get this fixed before 7-RELEASE ?


--- /usr/src/sys/conf/kern.mk.orig      2007-05-24 21:53:42.000000000 +0000
+++ /usr/src/sys/conf/kern.mk   2007-11-17 12:10:28.000000000 +0000
@@ -70,7 +70,7 @@
 #
 .if ${MACHINE_ARCH} == "amd64"
 CFLAGS+=       -mcmodel=kernel -mno-red-zone \
-               -mfpmath=387 -mno-sse -mno-sse2 -mno-mmx -mno-3dnow \
+               -mfpmath=387 -mno-sse -mno-sse2 -mno-sse3 -mno-mmx -mno-3dnow \
                -msoft-float -fno-asynchronous-unwind-tables
 INLINE_LIMIT?= 8000
 .endif

Attachment: pgpFOlFRUBVYc.pgp
Description: PGP signature

Reply via email to