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

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 19:59:57 PDT 2017

Previous message: [PATCH] D38944: [GVN] Handle removal of first implicit CF instruction correctly
Next message: [PATCH] D37460: [GVN] Prevent LoadPRE from hoisting across instructions that don't pass control flow to successors
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

mkazantsev added a comment.

Yes, I agree. In the current version of the patch these facts are enforced by asserts. Thanks for clarification!


https://reviews.llvm.org/D38944

Previous message: [PATCH] D38944: [GVN] Handle removal of first implicit CF instruction correctly
Next message: [PATCH] D37460: [GVN] Prevent LoadPRE from hoisting across instructions that don't pass control flow to successors
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list