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

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp:2956
   case ISD::BUILD_VECTOR:
+    if (Op.getValueType().isScalableVector())
+      break;
----------------
Is this possible? A BUILD_VECTOR with scalable-vector value type?


================
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;
----------------
Can EXTRACT_VECTOR_ELT produce a scalable vector? I'd have thought the existing check was sufficient.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137140