[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 Aug 23 13:20:14 PDT 2016


>
>
> While implementing caching is possible, I think we should avoid
> reinventing the wheel in memorySSA, but to keep the fix as simple as
> possible.  The new limit can be adjusted higher as its meaning has
> changed.
>

I'd be more okay with your last argument if you were going to commit to
rewriting this pass to MemorySSA.
:)

If we have no-one committed to moving the pass, and no plans to implement
caching, i guess my answer is "hope that someone will move this to
MemorySSA one day is not a strategy".
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160823/a7f70c1a/attachment.html>


More information about the llvm-commits mailing list