[PATCH] D28934: Write a new SSAUpdater

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 04:18:05 PST 2017


hfinkel added a comment.

In https://reviews.llvm.org/D28934#651323, @sanjoy wrote:

> I've never really heard people complain about our current `SSAUpdater` (about it being too slow or difficult to use) -- well except in this review :).  Is there a specific problem you're trying to address with this?


Do you find it easy to use? Perhaps I just haven't s studied it in sufficient detail, but I don't. Moreover, I think that's. our informal project policy has tended toward, "use this only if there's no other way." I'll admit never having thought about rewriting it, or if there is a better algorithm, but I think this is certainly an area in which potential improvements should be considered.


https://reviews.llvm.org/D28934





More information about the llvm-commits mailing list