This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Rename symbols types in preparation to adding wasm globals. NFC.
ClosedPublic

Authored by sbc100 on Feb 19 2018, 11:47 AM.

Details

Summary

Purely a rename, no functional change.

We want to use GlobalSymbol to represent real wasm globals and
DataSymbol for pointers to things in linear memory (what ELF would
call STT_OBJECT).

This reduces the size the patch to add the explicit symbol table
which is coming soon!

Diff Detail

Repository
rL LLVM

Event Timeline

sbc100 created this revision.Feb 19 2018, 11:47 AM
sbc100 retitled this revision from [WebAssembly] Rename symbols types in preparation to adding wasm globals to [WebAssembly] Rename symbols types in preparation to adding wasm globals. NFC..
sbc100 edited the summary of this revision. (Show Details)
ruiu accepted this revision.Feb 19 2018, 8:37 PM

LGTM

This revision is now accepted and ready to land.Feb 19 2018, 8:37 PM
ncw accepted this revision.Feb 20 2018, 5:09 AM
This revision was automatically updated to reflect the committed changes.