[PATCH] D118297: [clang] add Diag -Wasm-volatile for implied volatile asm stmts

[Segher Boessenkool via Phabricator via cfe-commits]

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