[PATCH] D112464: [x86] limit vector increment fold to allow load folding

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:5042
 
-static bool MayFoldLoad(SDValue Op, bool AssumeSingleUse = false) {
+bool X86::mayFoldLoad(SDValue Op, bool AssumeSingleUse) {
   return (AssumeSingleUse || Op.hasOneUse()) && ISD::isNormalLoad(Op.getNode());
----------------
Not related to this patch, but this function is incorrect for SSE which requires alignment or a subtarget feature that disables the alignment check. Are all existing uses scalar or AVX only?


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

https://reviews.llvm.org/D112464