Page MenuHomePhabricator

[Alignment][NFC] Use Align with TargetLowering::setMinFunctionAlignment
ClosedPublic

Authored by gchatelet on Thu, Sep 5, 9:16 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

gchatelet created this revision.Thu, Sep 5, 9:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Sep 5, 9:16 AM
gchatelet marked an inline comment as done.Thu, Sep 5, 9:18 AM
gchatelet added inline comments.
llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
123 ↗(On Diff #218937)

This one seems suspicious, it looks like it should be 2 byte aligned.

courbet accepted this revision.Fri, Sep 6, 5:13 AM
courbet added inline comments.
llvm/lib/Target/RISCV/RISCVISelLowering.cpp
201 ↗(On Diff #218937)

const ?

llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
123 ↗(On Diff #218937)

Let's submit the NFC change and create a separate patch for the semantic change.

This revision is now accepted and ready to land.Fri, Sep 6, 5:13 AM
gchatelet updated this revision to Diff 219074.Fri, Sep 6, 5:45 AM
gchatelet marked 2 inline comments as done.
  • Added const
This revision was automatically updated to reflect the committed changes.