[PATCH] D32798: NFC: refactor replaceDominatedUsesWith

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 14:38:02 PDT 2017


Prazek added a comment.

In https://reviews.llvm.org/D32798#745017, @davide wrote:

> I'm fine with this refactoring, but I'd like to understand what's your long(er) term plan for this first?


I am not sure how it will go, because I've found the flaw in my idea when writing this. Check out my recent post on mailing list.
Anyway, even if I won't gonna change this code later, I think it make sense to do the refactor.


https://reviews.llvm.org/D32798





More information about the llvm-commits mailing list