This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] MC: Use existing MCSymbol.Index field rather than inventing extra mapping
ClosedPublic

Authored by sbc100 on May 7 2018, 3:51 PM.

Details

Summary

MCSymbol has getIndex/setIndex which are implementation defined
and on other platforms are used to store the symbol table
index. It makes sense to use this rather than invent a new
mapping.

Diff Detail

Repository
rL LLVM

Event Timeline

sbc100 created this revision.May 7 2018, 3:51 PM
sbc100 updated this revision to Diff 145581.May 7 2018, 3:58 PM
  • revert
sbc100 added a reviewer: ncw.May 7 2018, 4:00 PM
sbc100 updated this revision to Diff 145582.May 7 2018, 4:02 PM
  • revert
sbc100 updated this revision to Diff 145593.May 7 2018, 4:52 PM
  • clang-format
dschuff accepted this revision.May 7 2018, 5:05 PM
This revision is now accepted and ready to land.May 7 2018, 5:05 PM
This revision was automatically updated to reflect the committed changes.