[PATCH] D52310: Set correct MMO offset on scalarized load pieces
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 21 21:41:36 PDT 2018
craig.topper added inline comments.
================
Comment at: lib/Target/X86/X86ISelLowering.cpp:19890
for (unsigned i = 0; i < NumLoads; ++i) {
+ unsigned NewAlign = MinAlign(Ld->getAlignment(), Offset);
+
----------------
I'm not sure if this is needed. getAlignment for loads defers to MachineMemOperand::getAlignment which calls MinAlign(getBaseAlignment(), Offset)
Repository:
rL LLVM
https://reviews.llvm.org/D52310
More information about the llvm-commits
mailing list