[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>
> On Tue, Apr 12, 2016 at 9:37 AM, Daniel Berlin <dberlin at dberlin.org>
>> On Tue, Apr 12, 2016 at 9:31 AM, Chandler Carruth <chandlerc at gmail.com>
>>> 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...
More information about the llvm-commits