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

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

2007-03-13 Thread Jeff Cohen
Changes in directory llvm/lib/Transforms/Utils: LowerSwitch.cpp updated: 1.35 -> 1.36 --- Log message: Unbreak VC++ build. Do not use identifiers starting with _ as they are reserved and can collide with system defined names. Windows defines _BB, for example. --- Diffs of the changes: (+3

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 >

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

2007-03-10 Thread Anton Korobeynikov
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 1 files changed, 114 i

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

2007-03-02 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Utils: LowerSwitch.cpp updated: 1.33 -> 1.34 --- Log message: Make sure debug code is not evaluated in non-debug case. --- Diffs of the changes: (+3 -2) LowerSwitch.cpp |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) Index: llvm/lib

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

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

2007-03-02 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Utils: LowerSwitch.cpp updated: 1.32 -> 1.33 --- Log message: 1. Sort switch cases using APInt safe comparison. 2. Make sure debug output of APInt values is safe for all bit widths. --- Diffs of the changes: (+2 -2) LowerSwitch.cpp |4 ++-- 1 fi

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

2006-12-16 Thread Bill Wendling
Changes in directory llvm/lib/Transforms/Utils: LowerSwitch.cpp updated: 1.28 -> 1.29 --- Log message: Added an automatic cast to "std::ostream*" etc. from OStream. We then can rework the hacks that had us passing OStream in. We pass in std::ostream* instead, check for null, and then dispatch t