[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

[Bob Wilson]

I'm still getting the same failures.  I'll look at updating X86.td.

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

> 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
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits