[llvm-commits] [llvm] r134641 - in /llvm/trunk: lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp lib/Target/X86/MCTargetDesc/X86MCTargetDesc.h lib/Target/X86/X86.td lib/Target/X86/X86Subtarget.cpp lib/Target/X86/X86Subtarget.h lib/Target/X86/X86Ta

[Evan Cheng]

r134741 should handle the non-x86 hosts. For the other cases, we should either fix tests or add specific CPUs to X86.td.

Evan

On Jul 8, 2011, at 2:06 PM, Evan Cheng wrote:

>> 
>> The bit of this change in X86Subtarget.cpp appears to break tests
>> anywhere that sys::getHostCPUName doesn't return the name of a known
>> 64-bit-capable x86 CPU (ARM, older x86 CPUs, very new x86 CPUS).
>> Please fix or revert.
> 
> I don't see how the previous logic would have worked either. Do you know which tests are failing? They probably need to explicitly set -mcpu.
> 
> Evan
> 
>> 
>> -Eli
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits