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

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 11:51:48 PST 2018


sbc100 added a comment.

Yes, I actually have another change to update the llvm side.   However, in light of your comments I guess I'll hold off.

Maybe once we drop the ELF output comletely we can  remove this part?

Also, how do you feel about passing the -mtripple to llc vs embedding the triple in the bitcode text?  I see both done and I'd like to be consistent.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42433





More information about the llvm-commits mailing list