This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Expand symbols shown by llvm-objdump --symbols
ClosedPublic

Authored by sbc100 on Feb 6 2019, 4:12 PM.

Diff Detail

Event Timeline

sbc100 created this revision.Feb 6 2019, 4:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2019, 4:12 PM
sbc100 updated this revision to Diff 185666.Feb 6 2019, 4:15 PM
  • fix build
rupprecht added inline comments.Feb 6 2019, 4:19 PM
tools/llvm-readobj/WasmDumper.cpp
207

Looks like there is a #define ENUM_ENTRY(X) pattern at the top of this file, can move this section there and use that pattern?

sbc100 updated this revision to Diff 185674.Feb 6 2019, 4:31 PM
  • feedback
rupprecht accepted this revision.Feb 6 2019, 4:56 PM
This revision is now accepted and ready to land.Feb 6 2019, 4:56 PM
This revision was automatically updated to reflect the committed changes.

Does this affect llvm-objdump or just llvm-readobj, or both (I don't see any tests here with llvm-objdump but the CL description mentions it)

sbc100 added a comment.Feb 6 2019, 5:18 PM

Sorry, typo in CL title.