Re: [llvm-commits] CVS: llvm/lib/Transforms/Utils/LowerSwitch.cpp

2007-03-13 Thread Chris Lattner
On Mar 10, 2007, at 8:50 AM, Nick Lewycky wrote: > >> +struct CaseRange { >> + Constant* Low; >> + Constant* High; >> + BasicBlock* BB; >> + >> + CaseRange(Constant* _Low = NULL, Constant* _High = NULL, >> +BasicBlock* _BB = NULL): >> +Low(_Low), High

Re: [llvm-commits] CVS: llvm/lib/Transforms/Utils/LowerSwitch.cpp

2007-03-10 Thread Nick Lewycky
Anton Korobeynikov wrote: > > Changes in directory llvm/lib/Transforms/Utils: > > LowerSwitch.cpp updated: 1.34 -> 1.35 > --- > Log message: > > Use range tests in LowerSwitch, where possible > > > --- > Diffs of the changes: (+114 -31) > > LowerSwitch.cpp | 145 >

Re: [llvm-commits] CVS: llvm/lib/Transforms/Utils/LowerSwitch.cpp

2007-03-02 Thread Chris Lattner
> 1. Sort switch cases using APInt safe comparison. Cool, > @@ -128,7 +128,7 @@ > >Case& Pivot = *(Begin + Mid); >DOUT << "Pivot ==> " > - << cast(Pivot.first)->getSExtValue() << "\n"; > + << cast(Pivot.first)->getValue().toStringSigned > (10) << "\n"; Please wrap this in a D