[PATCH] D15537: limit the number of instructions per block examined by dead store elimination
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 11 10:18:18 PDT 2016
tejohnson added a comment.
In http://reviews.llvm.org/D15537#397257, @davidxl wrote:
> yes -- MemSSA will get there. Short term workaround might also be needed :)
I think we need a stopgap fix for this non-linear compile time issue. I don't think waiting until it is reimplemented to use MemorySSA is a good strategy given the extreme compile time issues, and I can't even find a switch to disable this pass.
BTW, the author had provided a test case on the mailing list (there are two threads for this patch):
http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20160201/330335.html
http://reviews.llvm.org/D15537
More information about the llvm-commits
mailing list