[PATCH] D110111: [WebAssembly] Add relaxed-simd feature

Ng Zhi An via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 22 09:27:05 PDT 2021


ngzhian added a comment.

In D110111#3013948 <https://reviews.llvm.org/D110111#3013948>, @tlively wrote:

> Nice! Thanks for writing this :D Do you know what happens when you actually try to compile some code with `-mrelaxed-simd`? I'm concerned that it will throw an error because the "relaxed-simd" target feature has not yet been defined in the backend (specifically in WebAssembly.td and in WebAssemblySubtarget.{h,cpp}).

It output "'+relaxed-simd' is not a recognized feature for this target (ignoring feature)" but didn't fail compilation. I fixed it by adding to WebAssembly.td and WebAssemblySubtarget.h, thanks!

> Edit: Here's the corresponding previous patch that added the target feature in the backend: https://reviews.llvm.org/D56501.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110111



More information about the cfe-commits mailing list