[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