Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/LoopUnroll.cpp

2007-03-02 Thread Chris Lattner
On Mar 2, 2007, at 3:40 PM, Reid Spencer wrote: > On Fri, 2007-03-02 at 15:35 -0800, Chris Lattner wrote: >> On Mar 2, 2007, at 3:31 PM, Reid Spencer wrote: >> >> >>> + // Guard against huge trip counts. This also guards against >>> assertions in >>> + // APInt from the use of getZExtValue, bel

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/LoopUnroll.cpp

2007-03-02 Thread Reid Spencer
On Fri, 2007-03-02 at 15:35 -0800, Chris Lattner wrote: > On Mar 2, 2007, at 3:31 PM, Reid Spencer wrote: > > > > + // Guard against huge trip counts. This also guards against > > assertions in > > + // APInt from the use of getZExtValue, below. > > + if (TripCountC->getValue().getActiveBits

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/LoopUnroll.cpp

2007-03-02 Thread Chris Lattner
On Mar 2, 2007, at 3:31 PM, Reid Spencer wrote: > + // Guard against huge trip counts. This also guards against > assertions in > + // APInt from the use of getZExtValue, below. > + if (TripCountC->getValue().getActiveBits() > 32) > return Changed; // More than 2^32 iterations??? > > +