asb added a comment.

One thing that's not ideal about this test case is that it still passes if you 
delete the changes to WebAssembly.cpp. At least for RISC-V, selecting the -mcpu 
also emits the relevant `-target-feature +foo` for features supported by that 
CPU, which doesn't seem to happen here. Haven't been able to trace through to 
see if that's due to us doing something obviously different in the RISC-V case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125728

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

Reply via email to