This is an archive of the discontinued LLVM Phabricator instance.

Compact EhSectionPiece from 32 bytes to 16 bytes.
ClosedPublic

Authored by ruiu on Sep 18 2017, 5:21 PM.

Details

Summary

EhSectionPiece used to have a pointer to a section, but that pointer was
mostly redundant because we almost always know what the section is without
using that pointer. This patch removes the pointer from the struct.

This patch also use uint32_t/int32_t instead of size_t to represent
offsets that are hardly be larger than 4 GiB. At the moment, I think it is
OK even if we cannot handle .eh_frame sections larger than 4 GiB.

Diff Detail

Repository
rL LLVM

Event Timeline

ruiu created this revision.Sep 18 2017, 5:21 PM
ruiu updated this revision to Diff 115906.Sep 19 2017, 2:39 PM
  • Do not rename varaibles in this patch
  • Rebase
This revision was automatically updated to reflect the committed changes.