This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Move phdr typedef to cpp file
ClosedPublic

Authored by urnathan on Apr 18 2023, 3:28 PM.

Details

Summary

This typedef is only used inside the RewriteInstance source file, let's not
expose it in the header file -- even if private.

This makes it easier to support an ELF32 ABI.

Diff Detail

Event Timeline

urnathan created this revision.Apr 18 2023, 3:28 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 18 2023, 3:28 PM
urnathan requested review of this revision.Apr 18 2023, 3:28 PM
maksfb accepted this revision.Apr 18 2023, 5:58 PM

LGTM. Please add NFC (or NFCI) to the title.

This revision is now accepted and ready to land.Apr 18 2023, 5:58 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 19 2023, 12:52 PM