[PATCH] D38944: [GVN] Handle removal of first implicit CF instruction correctly

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 08:01:01 PDT 2017


dberlin added a comment.

Both of the deletions are in PRE, after it has been successful.  If either the inserted or the deleted instructions were implicit control flow, PRE should have been unsafe.  It's probably worth noting this in a comment


https://reviews.llvm.org/D38944





More information about the llvm-commits mailing list