[PATCH] D86431: [SVE][CodeGen] Fix up warnings in sve-split-insert/extract tests

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 17:07:26 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:2036
+    // else we can do at this point.
+    if (KnownIdx == -1 && SrcEC.Scalable)
+      return false;
----------------
Please bail out here for scalable vectors even if KnownIdx is in range.  The rest of the demanded bits code code doesn't expect scalable vectors at the moment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86431



More information about the llvm-commits mailing list