Re: [llvm-commits] [llvm] r44082 - /llvm/trunk/include/llvm/ADT/SparseBitVector.h

2007-11-14 Thread Daniel Berlin
On 11/13/07, Owen Anderson <[EMAIL PROTECTED]> wrote: > On Nov 13, 2007, at 5:04 PM, Hartmut Kaiser wrote: > > -return iterator(this, ~0); > > +return iterator(this, true); > > I'm not sure this is right. Is "true" guaranteed to be all-ones when > cast to an integer, or just to contain at

Re: [llvm-commits] [llvm] r44082 - /llvm/trunk/include/llvm/ADT/SparseBitVector.h

2007-11-13 Thread Owen Anderson
On Nov 13, 2007, at 5:04 PM, Hartmut Kaiser wrote: -return iterator(this, ~0); +return iterator(this, true); I'm not sure this is right. Is "true" guaranteed to be all-ones when cast to an integer, or just to contain at least one 1 bit? If the latter, then this patch changes the m

[llvm-commits] [llvm] r44082 - /llvm/trunk/include/llvm/ADT/SparseBitVector.h

2007-11-13 Thread Hartmut Kaiser
Author: hkaiser Date: Tue Nov 13 17:04:06 2007 New Revision: 44082 URL: http://llvm.org/viewvc/llvm-project?rev=44082&view=rev Log: Fixed a strange construct. Please review. Modified: llvm/trunk/include/llvm/ADT/SparseBitVector.h Modified: llvm/trunk/include/llvm/ADT/SparseBitVector.h URL: