[llvm] [AArch64] Remove AArch64TargetLowering::shouldExpandPartialReductionIntrinsic (NFC) (PR #159546)

Benjamin Maxwell via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 18 03:16:11 PDT 2025


https://github.com/MacDue created https://github.com/llvm/llvm-project/pull/159546

The AArch64 implementation now matches the base implementation.

>From 48aa4ce53a178973210f265aad2f75c53b574948 Mon Sep 17 00:00:00 2001
From: Benjamin Maxwell <benjamin.maxwell at arm.com>
Date: Wed, 17 Sep 2025 14:51:39 +0000
Subject: [PATCH] [AArch64] Remove
 AArch64TargetLowering::shouldExpandPartialReductionIntrinsic (NFC)

The AArch64 implemetation now matches the base implemetation.
---
 llvm/lib/Target/AArch64/AArch64ISelLowering.cpp | 7 -------
 llvm/lib/Target/AArch64/AArch64ISelLowering.h   | 3 ---
 2 files changed, 10 deletions(-)

diff --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
index fc3efb072d57b..3eda0a972802c 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -2182,13 +2182,6 @@ bool AArch64TargetLowering::shouldExpandGetActiveLaneMask(EVT ResVT,
   return false;
 }
 
-bool AArch64TargetLowering::shouldExpandPartialReductionIntrinsic(
-    const IntrinsicInst *I) const {
-  assert(I->getIntrinsicID() == Intrinsic::vector_partial_reduce_add &&
-         "Unexpected intrinsic!");
-  return true;
-}
-
 bool AArch64TargetLowering::shouldExpandCttzElements(EVT VT) const {
   if (!Subtarget->isSVEorStreamingSVEAvailable())
     return true;
diff --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.h b/llvm/lib/Target/AArch64/AArch64ISelLowering.h
index ff073d3eafb1f..97bc4b7a9c5c2 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.h
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.h
@@ -523,9 +523,6 @@ class AArch64TargetLowering : public TargetLowering {
 
   bool shouldExpandGetActiveLaneMask(EVT VT, EVT OpVT) const override;
 
-  bool
-  shouldExpandPartialReductionIntrinsic(const IntrinsicInst *I) const override;
-
   bool shouldExpandCttzElements(EVT VT) const override;
 
   bool shouldExpandVectorMatch(EVT VT, unsigned SearchSize) const override;



More information about the llvm-commits mailing list