[PATCH] D118297: [clang] add Diag -Wasm-volatile for implied volatile asm stmts
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 31 13:53:44 PST 2022
MaskRay added a comment.
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")`.
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