[PATCH] D153445: [SVE ACLE] Remove DAG combines that are no longer relevant.

Jolanta Jensen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 02:17:25 PDT 2023


jolanta.jensen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:18922-18923
     return LowerSVEIntrinsicEXT(N, DAG);
   case Intrinsic::aarch64_sve_mul:
-    return convertMergedOpToPredOp(N, AArch64ISD::MUL_PRED, DAG);
+    return SDValue();
   case Intrinsic::aarch64_sve_mul_u:
----------------
paulwalker-arm wrote:
> Rather than returning `SDValue();` you should be able to remove the case statement entirely for the affected intrinsic IDs.
Fixed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153445



More information about the llvm-commits mailing list