[PATCH] D37460: [GVN] Prevent LoadPRE from hoisting through guards
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 11 17:14:23 PDT 2017
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
More information about the llvm-commits
mailing list