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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a subscriber: reames.
efriedma added a comment.

We consider volatile loads and stores as potentially trapping as a courtesy to our users.  Strictly speaking, it's undefined behavior in C, but people sometimes intentionally trigger SIGSEGV using volatile loads, so we don't want to cause an unrelated crash before that happens.  Breaking that testcase should be fine.


https://reviews.llvm.org/D37460