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

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

I think this makes sense. LGTM, one nit.



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


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

https://reviews.llvm.org/D74645