[PATCH] D85795: [WebAssembly][AsmParser] Name missing features in error message

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 17:19:33 PDT 2020


tlively added inline comments.


================
Comment at: llvm/test/MC/WebAssembly/missing-features.s:6
+# CHECK: error: instruction requires: simd128
+needs_simd:
+    .functype needs_simd () -> (v128)
----------------
dschuff wrote:
> might as well add one that requires more than one feature?
I don't think we actually have any such instructions right now. But I'd rather keep the loop to be robust against adding any multi-feature instructions in the future.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85795



More information about the llvm-commits mailing list