Page MenuHomePhabricator

[Alignment] Move OffsetToAlignment to Alignment.h
ClosedPublic

Authored by gchatelet on Sep 12 2019, 7:34 AM.

Event Timeline

gchatelet created this revision.Sep 12 2019, 7:34 AM
Herald added a reviewer: alexshap. · View Herald Transcript
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
courbet added inline comments.Sep 12 2019, 7:56 AM
llvm/lib/Target/ARM/ARMConstantIslandPass.cpp
1024

this statement is now useless.

llvm/lib/Target/Mips/MipsSERegisterInfo.cpp
216

what about:

const llvm::Align OffsetAlign =
        llvm::Align(llvm::Align(getLoadStoreOffsetAlign(MI.getOpcode()))));

:)

gchatelet updated this revision to Diff 219923.Sep 12 2019, 8:02 AM
gchatelet marked an inline comment as done.
  • Address comments
gchatelet marked an inline comment as done.Sep 12 2019, 8:02 AM
courbet accepted this revision.Sep 12 2019, 8:07 AM
This revision is now accepted and ready to land.Sep 12 2019, 8:07 AM
This revision was automatically updated to reflect the committed changes.