[llvm-commits] [llvm] r167410 - /llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

[Richard Smith]

On Mon, Nov 5, 2012 at 2:25 PM, Craig Topper <craig.topper at gmail.com> wrote:
> Doesn't LLVM normally use capital U for the suffix?

Ah, so it does (it's about 20x more common than the lowercase suffix).
Nadav fixed this in r167421.

> On Mon, Nov 5, 2012 at 2:01 PM, Richard Smith <richard-llvm at metafoo.co.uk>
> wrote:
>>
>> Author: rsmith
>> Date: Mon Nov  5 16:01:44 2012
>> New Revision: 167410
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=167410&view=rev
>> Log:
>> Suppress signed/unsigned comparison warning.
>>
>> Modified:
>>     llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
>>
>> Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=167410&r1=167409&r2=167410&view=diff
>>
>> ==============================================================================
>> --- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
>> +++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Mon Nov  5 16:01:44 2012
>> @@ -17555,10 +17555,10 @@
>>
>>  unsigned
>>  X86VectorTargetTransformInfo::getVectorInstrCost(unsigned Opcode, Type
>> *Val,
>> -                                    unsigned Index) const {
>> +                                                 unsigned Index) const {
>>    assert(Val->isVectorTy() && "This must be a vector type");
>>
>> -  if (Index != -1) {
>> +  if (Index != -1u) {
>>      // Legalize the type.
>>      std::pair<unsigned, MVT> LT =
>>      getTypeLegalizationCost(Val->getContext(), TLI->getValueType(Val));
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
>
>
> --
> ~Craig