[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