[PATCH] D96097: [TargetLowering] Use Align in allowsMisalignedMemoryAccesses.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 19:08:32 PST 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:1060
           TLI.allowsMisalignedMemoryAccesses(
-              VT, DstAS, Op.isFixedDstAlign() ? Op.getDstAlign().value() : 0,
+              VT, DstAS, Op.isFixedDstAlign() ? Op.getDstAlign() : Align(1),
               MachineMemOperand::MONone, &Fast) &&
----------------
Note, this previously passed 0 and now passes 1.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96097/new/

https://reviews.llvm.org/D96097



More information about the llvm-commits mailing list