[PATCH] D118376: [x86] try harder to scalarize a vector load with extracted integer op uses

Phoebe Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 27 17:36:53 PST 2022


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

Thanks for the improvement! LGTM.



================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:43086-43088
+  // Note: Unlike the related fold for this in DAGCombiner, this is not limited
+  //       to a single-use of the loaded vector. For the reasons above, we
+  //       expect this to be profitable even if it creates an extra load.
----------------
Add a regression test for it?


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

https://reviews.llvm.org/D118376



More information about the llvm-commits mailing list