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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

The volatile load thing never really got discussed in depth, as far as I remember... I wanted to be conservative because volatile memory access can be weird, but I guess my interpretation in isGuaranteedToTransferExecutionToSuccessor() is a little suspect given the way alias analysis treats volatile operations.


https://reviews.llvm.org/D37460