[PATCH] D110111: [WebAssembly] Add relaxed-simd feature
Ng Zhi An via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list