[PATCH] D48744: [WebAssembly] Remove ELF file support.

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 2 10:16:54 PDT 2018


aardappel added inline comments.


================
Comment at: lib/MC/MCParser/AsmParser.cpp:700
+    // same for now.
     PlatformParser.reset(createELFAsmParser());
     break;
----------------
sbc100 wrote:
> aardappel wrote:
> > This one should be renamed, since there is now only one MCAsmParserExtension for wasm, and its implementation is actually kinda ELF-y, but it is wha we'll use for wasm mode going forward.
> Maybe a separate change?
that's fine with me.


Repository:
  rL LLVM

https://reviews.llvm.org/D48744





More information about the llvm-commits mailing list