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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to