[llvm] r202295 - Silencing an MSVC signed comparison warning.

David Blaikie dblaikie at gmail.com
Wed Feb 26 12:55:52 PST 2014


On Wed, Feb 26, 2014 at 12:22 PM, Aaron Ballman <aaron at aaronballman.com>wrote:

> Author: aaronballman
> Date: Wed Feb 26 14:22:20 2014
> New Revision: 202295
>
> URL: http://llvm.org/viewvc/llvm-project?rev=202295&view=rev
> Log:
> Silencing an MSVC signed comparison warning.
>

Do we have a bug tracking Clang's implementation of this? (or turning it on
for the LLVM build if it's already implemented)

(either that, or can we turn off the MSVC warning if it's not considered
high value?)


>
> Modified:
>     llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp
>
> Modified: llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp?rev=202295&r1=202294&r2=202295&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp (original)
> +++ llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp Wed Feb 26 14:22:20 2014
> @@ -189,7 +189,7 @@ bool PPCCTRLoops::runOnFunction(Function
>
>  static bool isLargeIntegerTy(bool Is32Bit, Type *Ty) {
>    if (IntegerType *ITy = dyn_cast<IntegerType>(Ty))
> -    return ITy->getBitWidth() > (Is32Bit ? 32 : 64);
> +    return ITy->getBitWidth() > (Is32Bit ? 32U : 64U);
>
>    return false;
>  }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140226/ea359e86/attachment.html>


More information about the llvm-commits mailing list