This is an archive of the discontinued LLVM Phabricator instance.

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

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

Details

Event Timeline

gchatelet created this revision.Sep 5 2019, 9:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 5 2019, 9:16 AM
gchatelet marked an inline comment as done.Sep 5 2019, 9:18 AM
gchatelet added inline comments.
llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
123

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

courbet accepted this revision.Sep 6 2019, 5:13 AM
courbet added inline comments.
llvm/lib/Target/RISCV/RISCVISelLowering.cpp
201–203

const ?

llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
123

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

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