[PATCH] D42433: [WebAssembly] Update wasm target triple now that its the default

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 12:23:08 PST 2018


dschuff added a comment.

I was going to say that most llc tests in LLVM don't use mtriple and that we should be consistent with that, but a quick grep suggests that something like 65% of the RUN lines with llc actually do contain `mtriple`. So I guess we could go either way. I think we still probably want to keep the triple in the bitcode (it is more convenient to run just llc, but only marginally so), in which case mtriple would be redundant. But I don't have strong opinions.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42433





More information about the llvm-commits mailing list