This is an archive of the discontinued LLVM Phabricator instance.

[Alignment] fix dubious min function alignment
ClosedPublic

Authored by gchatelet on Sep 6 2019, 6:01 AM.

Details

Summary

This was discovered while introducing the llvm::Align type.
The original setMinFunctionAlignment used to take alignment as log2, looking at the comment it seems like instructions are to be 2-bytes aligned and not 4-bytes aligned.

Diff Detail

Repository
rL LLVM

Event Timeline

gchatelet created this revision.Sep 6 2019, 6:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 6 2019, 6:01 AM
uweigand accepted this revision.Sep 6 2019, 6:17 AM

LGTM.

This revision is now accepted and ready to land.Sep 6 2019, 6:17 AM
This revision was automatically updated to reflect the committed changes.