[PATCH] D47745: Simplified WebAssemblyAsmBackend by removing explicit ELF variant.

[Derek Schuff via Phabricator via llvm-commits]

dschuff added inline comments.


================
Comment at: lib/Target/WebAssembly/MCTargetDesc/WebAssemblyAsmBackend.cpp:125
+  return IsELF ? createWebAssemblyELFObjectWriter(OS, Is64Bit, 0)
+               : createWebAssemblyWasmObjectWriter(OS, Is64Bit);
 }
----------------
sbc100 wrote:
> IIRC we could probably abort here in the elf case as I think ELF is only used to produce .s format files, not object files.
> 
> I guess that might be a separate/larger change so this LGTM as is keep the change small.
Yeah we should be able to remove the ELF object writer too.


Repository:
  rL LLVM

https://reviews.llvm.org/D47745