[PATCH] D88396: [X86] Replace aligned vector move with unaligned move when avx is enabled.

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/lib/Target/X86/X86UnalignedVectorMoves.cpp:93
+    for (MachineInstr &MI : MBB)
+      Changed |= (MovapsToMovupsImpl(MI) || MovapdToMovupdImpl(MI) ||
+                  MovdqaToMovdquImpl(MI));
----------------
craig.topper wrote:
> Why do we need 3 separate functions?
Separating into 3 function looks clearer to me. I can merge them into 1 switch clause and add 3 comments for the code. Do you prefer merge?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88396