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

2008-02-17 Thread 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; +

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

2008-02-17 Thread Bill Wendling
Dan submitted a fix last week. -bw On Feb 17, 2008, at 2:16 PM, Evan Cheng wrote: 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

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

2008-02-13 Thread Dan Gohman
Author: djg Date: Wed Feb 13 15:11:05 2008 New Revision: 47086 URL: http://llvm.org/viewvc/llvm-project?rev=47086view=rev Log: Add countTrailingOnes member functions to APInt. Modified: llvm/trunk/include/llvm/ADT/APInt.h llvm/trunk/lib/Support/APInt.cpp Modified:

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

2008-02-13 Thread Bill Wendling
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