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

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 29 15:35:36 PDT 2018


aardappel created this revision.
aardappel added reviewers: sbc100, dschuff.
Herald added subscribers: llvm-commits, kristina, dexonsmith, aheejin, jgravelle-google.

The assembler was able to assemble and then dump back to .s, but
was failing to parse certain directives necessary for valid .o
output:

- .type directives are now recognized to distinguish function symbols and others.
- .size is now parsed to provide function size.

Also makes a small change in StringSwitch.h for some missing
functionality (being able to detect no match).


Repository:
  rL LLVM

https://reviews.llvm.org/D53842

Files:
  include/llvm/ADT/StringSwitch.h
  include/llvm/MC/MCSymbolWasm.h
  lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp
  lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp
  test/MC/WebAssembly/basic-assembly.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53842.171596.patch
Type: text/x-patch
Size: 9689 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181029/0e3ad31d/attachment.bin>


More information about the llvm-commits mailing list