[llvm-commits] [llvm] r63495 - in /llvm/trunk: lib/Target/X86/X86CallingConv.td lib/Target/X86/X86ISelLowering.cpp lib/Target/X86/X86Subtarget.cpp test/CodeGen/X86/2009-01-25-NoSSE.ll test/CodeGen/X86/nosse-varargs.ll

[Dan Gohman]

Hello Edwin,



Dan

On Feb 1, 2009, at 10:15 AM, Torok Edwin wrote:

>
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- llvm/trunk/lib/Target/X86/X86Subtarget.cpp (original)
> +++ llvm/trunk/lib/Target/X86/X86Subtarget.cpp Sun Feb  1 12:15:56  
> 2009
> @@ -331,7 +331,7 @@
>   // are enabled.  These are available on all x86-64 CPUs.
>   if (Is64Bit) {
>     HasX86_64 = true;
> -#if 1
> +#if 0
>      if (X86SSELevel < SSE2)
>        X86SSELevel = SSE2;
> #endif


Can you fix the comment immediately above this to reflect
what the code now does?

Also, without this code, does -march=x86-64 default to having
SSE2 disabled (or dependent on the host)? Since all known
x86-64 hardware supports SSE2, it's convenient to have SSE2
enabled by default on x86-64.

Thanks,

Dan