[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