[PATCH] D87149: [InstCombine] erase instructions leading up to unreachable

Wed Sep 9 12:15:09 PDT 2020

nikic added a comment.

In D87149#2264099 <https://reviews.llvm.org/D87149#2264099>, @chandlerc wrote:

> Regarding access to volatile memory...
>
> If volatile memory access is known to not trap, then the guidance on LLVM's own blog is wrong:
> https://blog.llvm.org/posts/2011-05-13-what-every-c-programmer-should-know/

This is a (non-normative) blog post from 2011. The requirements on volatile operations have been clarified in D53184 <https://reviews.llvm.org/D53184> after a llvm-dev discussion. Of course we can re-evaluate this decision, in which case a LangRef patch needs to be proposed and a new RFC on llvm-dev started.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87149/new/

https://reviews.llvm.org/D87149