Page MenuHomePhabricator

[Alignment] fix dubious min function alignment
ClosedPublic

Authored by gchatelet on Fri, Sep 6, 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.Fri, Sep 6, 6:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptFri, Sep 6, 6:01 AM
uweigand accepted this revision.Fri, Sep 6, 6:17 AM

LGTM.

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