[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/

[Stepan Dyatkovskiy]

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 
in reply.
I carefully checked it again and again.
Though, if you found something changes I made, I'm ready for discussing.

-Stepan.

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?
>
> -eric