[PATCH] D118297: [clang] add Diag -Wasm-volatile for implied volatile asm stmts
Segher Boessenkool via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 31 14:23:39 PST 2022
segher added a comment.
In D118297#3285571 <https://reviews.llvm.org/D118297#3285571>, @MaskRay wrote:
> The `volatile` qualifier is implied but makes the intention explicit. I agree that the user should not be punished (-Wasm-volatile) by writing `asm volatile("int3")` instead of `asm("int3")`.
Not punished no. But it is useful education (like all the other "statement without effect"
etc. warnings).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118297/new/
https://reviews.llvm.org/D118297
More information about the cfe-commits
mailing list