[PATCH] D103880: [SDAG] Fix pow2 assumption when splitting vectors

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 9 22:01:44 PDT 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:19366
+      if (TLI.isTypeLegal(SplitVT) &&
+          SplitSize + SplitVT.getVectorMinNumElements() <=
+              InVT.getVectorMinNumElements()) {
----------------
I think you can use getVectorNumElements(). These should be fixed vectors I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103880



More information about the llvm-commits mailing list