[PATCH] D74645: Utils: Always alignment when expanding mem intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm closed this revision.
arsenm marked 2 inline comments as done.
arsenm added a comment.

b0bdb186f506d7fa2686f1c09bb8926b6adeb91d <https://reviews.llvm.org/rGb0bdb186f506d7fa2686f1c09bb8926b6adeb91d>



================
Comment at: llvm/lib/Transforms/Utils/LowerMemIntrinsics.cpp:64
+    Align PartDstAlign(MinAlign(DstAlign, PartSize));
+    Align PartSrcAlign(MinAlign(SrcAlign, PartSize));
+
----------------
jdoerfert wrote:
> Isn't `PartSize == LoopOpSize` ?
Yes


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

https://reviews.llvm.org/D74645