[PATCH] D135564: [AArch64-SVE]: Force generating code compatible to streaming mode.

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:22476-22477
     // "cast" fixed length vector to a scalable vector.
-    assert(useSVEForFixedLengthVectorVT(V.getValueType()) &&
+    assert(useSVEForFixedLengthVectorVT(
+               V.getValueType(), Subtarget->forceStreamingCompatibleSVE()) &&
            "Only fixed length vectors are supported!");
----------------
As with the above change can this be `V.getValueType().isFixedLengthVector() && isTypeLegal(V.getValueType()) &&`?


================
Comment at: llvm/test/CodeGen/AArch64/sve-streaming-fixed-length-int-shifts.ll:1-5
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -force-streaming-compatible-sve < %s | FileCheck %s
+
+target triple = "aarch64-unknown-linux-gnu"
+
----------------
Please rename this file `sve-streaming-mode-fixed-length-int-shifts.ll` to match the same format as the others.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135564