[llvm-commits] PATCH: instcombine switch on select of constants to br

Cameron Zwarich zwarich at apple.com
Tue Mar 8 22:38:53 PST 2011


On Feb 27, 2011, at 7:01 PM, Nick Lewycky wrote:

> Frits van Bommel wrote:
>> Since there has been no progress here for over six weeks, I updated
>> this patch myself.
>> Does anyone mind if I commit this version?
> 
> I kinda wish it didn't work this way. Here's what I have in mind:
>   1. fix llvm.org/PR774 (make the default case optional)
>   2. one optimization which finds switches that have two destinations 
> and turns them into branches
>   3. one optimization which sees select (and other things) feeding into 
> a switch and removes the impossible cases from the switch, possibly 
> including the default case.

This sounds relevant to my interests... Have you estimated how much work it would take?

Cameron



More information about the llvm-commits mailing list