This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Unify concepts of disgarded and non-live input chunks. NFC.
ClosedPublic

Authored by sbc100 on Feb 13 2018, 11:29 AM.

Details

Summary

It seems redundant to store this information twice. None of the
locations where this bit is checks cares about the distinction.

This simplifies the code and avoids having to reason about two
different concepts of aliveness.

Diff Detail

Repository
rL LLVM

Event Timeline

sbc100 created this revision.Feb 13 2018, 11:29 AM
sbc100 updated this revision to Diff 134090.Feb 13 2018, 11:31 AM
  • update comment
Harbormaster completed remote builds in B14936: Diff 134090.
sbc100 edited the summary of this revision. (Show Details)Feb 13 2018, 11:33 AM
sbc100 added reviewers: ruiu, ncw.
sbc100 edited the summary of this revision. (Show Details)
ruiu accepted this revision.Feb 13 2018, 11:37 AM

LGTM

This revision is now accepted and ready to land.Feb 13 2018, 11:37 AM
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.
ncw added a comment.Feb 14 2018, 4:16 AM

Cool, looks like it should work.

NB. Discarded/Live were distinguished based on the ELF and COFF implementations, so maybe we'll find a reason to bring the distinction back later - or maybe those can be simplified too, just to use the Live bit?