[PATCH] D137140: [SDAG] Allow scalable vectors in ComputeKnownBits

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 07:50:12 PDT 2022


reames added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp:2956
   case ISD::BUILD_VECTOR:
+    if (Op.getValueType().isScalableVector())
+      break;
----------------
frasercrmck wrote:
> Is this possible? A BUILD_VECTOR with scalable-vector value type?
I vaguely think I've seen this in debug output before, but converting it into an assert didn't catch any failures, and I don't remember the case.  Let's make it an assert and see if anything falls out.  


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp:3645
     // computeKnownBits not yet implemented for scalable vectors.
-    if (VecVT.isScalableVector())
+    if (Op.getValueType().isScalableVector() || VecVT.isScalableVector())
       break;
----------------
frasercrmck wrote:
> Can EXTRACT_VECTOR_ELT produce a scalable vector? I'd have thought the existing check was sufficient.
Good catch


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

https://reviews.llvm.org/D137140



More information about the llvm-commits mailing list