[PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 18 13:05:07 PDT 2019

Previous message: [PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate
Next message: [PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

craig.topper added a comment.

Shouldn't the definition in BuiltinsWebAssembly.def be updated to include an 'I' in the type string so that this will be properly diagnosed in the frontend?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59448/new/

https://reviews.llvm.org/D59448

Previous message: [PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate
Next message: [PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the cfe-commits mailing list