[PATCH] D28168: [LV] Update dominator tree before fixing external IV users

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 3 13:44:58 PST 2017


mkuper added a comment.

My gut feeling is that we really want to update DT as we go.
Even if all current uses of transform() except this one are safe (and I'm not at all sure about that), leaving this as is sounds very brittle, since any additional use of transform() can potentially break it.


https://reviews.llvm.org/D28168





More information about the llvm-commits mailing list