[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