erichkeane added a comment. This seems logical to me. The comment needs some love, so I made my best suggestion.
================ Comment at: clang/lib/CodeGen/CGExpr.cpp:1855 + // If this is a simple volatile store to a null pointer (and said null + // pointre is not defined), then we have a common idiom "please crash now". + // However, that is not the semantics LLVM IR provides, volatile operations ---------------- ================ Comment at: clang/lib/CodeGen/CGExpr.cpp:1856 + // pointre is not defined), then we have a common idiom "please crash now". + // However, that is not the semantics LLVM IR provides, volatile operations + // are not defined as trapping in LLVM IR, so this store will be ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105728/new/ https://reviews.llvm.org/D105728 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
