[PATCH] D15537: limit the number of instructions per block examined by dead store elimination

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 12 09:46:21 PDT 2016


On Tue, Apr 12, 2016 at 9:42 AM, Xinliang David Li <davidxl at google.com>
wrote:

>
>
> On Tue, Apr 12, 2016 at 9:37 AM, Daniel Berlin <dberlin at dberlin.org>
> wrote:
>
>>
>>
>> On Tue, Apr 12, 2016 at 9:31 AM, Chandler Carruth <chandlerc at gmail.com>
>> wrote:
>>
>>> chandlerc added a subscriber: chandlerc.
>>> chandlerc added a comment.
>>>
>>> FWIW, I think it is reasonable to workaround quadratic compile time
>>> problems with temporary limits until a well scaling algorithm lands.
>>>
>>
>> FWIW: I would agree, if i had any belief or hope that it would be
>> temporary ;)
>>
>
> MemSSA is in progress so it is definitely temporary.
>
>
I made my point at this point, i'm not going to argue further about it. It
looks like Chandler is willing to review and approve this, so there is a
clear path forward for folks.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160412/e0382d41/attachment.html>


More information about the llvm-commits mailing list