[PATCH] D37460: [GVN] Prevent LoadPRE from hoisting through guards

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 7 07:37:06 PDT 2017


dberlin added a comment.

Of course. Just in case you try to use the code, i'm pretty sure      if (OrderedInstructions->dominates(*Iter, guard))
 should be      if (OrderedInstructions->dominates(guard, *Iter))

(At least one of the calls is reversed)


https://reviews.llvm.org/D37460





More information about the llvm-commits mailing list