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

[Craig Topper]

Doesn't LLVM normally use capital U for the suffix?

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121105/854f4c4d/attachment.html>