[llvm-commits] [llvm] r47086 - in /llvm/trunk: include/llvm/ADT/APInt.h lib/Support/APInt.cpp

[Evan Cheng]

Ping. Is this fixed? Does it affect build with -Werror set?

Evan

On Feb 13, 2008, at 5:22 PM, Bill Wendling wrote:

> Hi Dan,
>
>> +uint32_t APInt::countTrailingOnes() const {
>> +  if (isSingleWord())
>> +    return std::min(uint32_t(CountTrailingOnes_64(VAL)), BitWidth);
>> +  uint32_t Count = 0;
>> +  uint32_t i = 0;
>> +  for (; i < getNumWords() && pVal[i] == -1; ++i)
>
> This compare leads to this warning:
>
> llvm[1]: Compiling APInt.cpp for Debug build
> /Volumes/Gir/devel/llvm/llvm.src/lib/Support/APInt.cpp: In member
> function 'uint32_t llvm::APInt::countTrailingOnes() const':
> /Volumes/Gir/devel/llvm/llvm.src/lib/Support/APInt.cpp:821: warning:
> comparison between signed and unsigned integer expressions
>
> Could you take a look at it please?
>
> -bw
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits