[PATCH] D53842: [WebAssembly] Parsing missing directives to produce valid .o

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 2 14:51:01 PDT 2018


aardappel marked an inline comment as done.
aardappel added inline comments.


================
Comment at: lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp:363
+        return Error("Cannot parse .size expression: ", Lexer.getTok());
+      WasmSym->setSize(Exp);
+    } else if (DirectiveID.getString() == ".globaltype") {
----------------
sbc100 wrote:
> What isn't `ELFAsmParser::ParseDirectiveSize` which then calls `getStreamer().emitELFSize` doing this already?
As discussed offline will do this in a followup, as it requires our own implementation of ELFAsmParser


Repository:
  rL LLVM

https://reviews.llvm.org/D53842





More information about the llvm-commits mailing list