[PATCH] D22305: [BasicAA] Strip phi nodes, when all incoming values are the same.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 15 13:08:37 PDT 2016


>
>
>
> Or do we need (2) because in general this kind of phi node is seen in many
>> programs? Or something else that I do not see....
>>
>
> That is hal's view.
>
> My view was that we should measure the impact of the change and then
> decide; but that if we have a use case for it, I'd rather it be inside the
> generic utility than a specific work-around in BasicAA.
>

Yes, sorry, was writing quickly, did not mean to imply anything else ;)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160715/ab577c81/attachment.html>


More information about the llvm-commits mailing list