[llvm-commits] [llvm] r149481 - in /llvm/trunk: include/llvm/ include/llvm/Analysis/ lib/Analysis/ lib/Bitcode/Writer/ lib/CodeGen/SelectionDAG/ lib/ExecutionEngine/Interpreter/ lib/Target/CBackend/ lib/Target/CppBackend/ lib/Transforms/IPO/ lib/Transforms/InstCombine/ lib/Transforms/Scalar/ lib/Transforms/Utils/ lib/VMCore/ tools/llvm-diff/
stpworld at narod.ru
Wed Feb 1 13:50:08 CST 2012
Hi Eric. I hope that it allows to change SwitchInst internals as we want
without fixing any 3rd side sources (llvm sources itself and llvm
clients). Instead I made it as pre-patch for PR1255: "Should enhance
LLVM switch instruction to take case ranges". Sorry for the long delay
I carefully checked it again and again.
Though, if you found something changes I made, I'm ready for discussing.
Eric Christopher wrote:
> On Jan 31, 2012, at 11:49 PM, Stepan Dyatkovskiy wrote:
>> SwitchInst refactoring.
>> The purpose of refactoring is to hide operand roles from SwitchInst
>> user (programmer). If you want to play with operands directly,
>> probably you will need lower level methods than SwitchInst ones
>> (TerminatorInst or may be User). After this patch we can reorganize
>> SwitchInst operands and successors as we want.
> Wait a minute.
> Why are you doing this? What do you hope to gain?
More information about the llvm-commits