[llvm-commits] [patch] Add a pass for duplicating indirectbr at the IL level

Bob Wilson bob.wilson at apple.com
Tue Aug 2 09:49:51 PDT 2011


On Aug 2, 2011, at 8:35 AM, Rafael Ávila de Espíndola wrote:
> 
>> The alternative is that we both go our own ways, end up with 3 or 4 tail
>> dup passes, and then have to throw one or more of them away. That would
>> be a shame, I think.
> 
> It is more of a shame to block work based on future expectations. While we have different opinions on which path is best, the only way to be sure is benchmark it. If I finish implementing my pass, get 1% in sunspider, I would be more than happy to replace it with yours (even if it needs an extra folding pass) if it got 2%.

OK, fine, I will confine my comments on your patch to issues that apply regardless of whether I make progress on my own project.  My goal was certainly not to block your work.  I just thought it would be good to coordinate our efforts.

I'm not going to have time today to write up any more details of my plans.  I'll still try to make it a priority.



More information about the llvm-commits mailing list