This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Adjust Shdr alignment
ClosedPublic

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

Details

Summary

Shdr's are not necessarily size 2^n -- that's an accident of ELF64_Shdr's contents.
There is no reason to align to that boundary if they are.

This fixes an assert fail for ELF32 (which requires more patches)

Diff Detail

Event Timeline

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

LGTM. Thanks.

This revision is now accepted and ready to land.Apr 18 2023, 5:56 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