[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