[PATCH] D80824: [CodeGen][SVE] Avoid scalarizing zero splat stores on scalable vectors.

Henry Kao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 8 11:35:29 PDT 2020


hkao13 marked an inline comment as done.
hkao13 added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:11973
+  // Avoid scalarizing zero splat stores for scalable vectors.
+  if (VT.isScalableVector() || !VT.isFixedLengthVector())
+    return SDValue();
----------------
sdesmalen wrote:
> I think you can remove the condition `|| !VT.isFixedLengthVector()`, because this function would otherwise break if it isn't.
Fixed, thanks for feedback.


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

https://reviews.llvm.org/D80824





More information about the llvm-commits mailing list