[llvm] d0a3820 - [NFC] Remove assert from AArch64TargetLowering::LowerCTTZ. (#193474)

via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 02:30:16 PDT 2026


Author: Paul Walker
Date: 2026-04-23T10:30:12+01:00
New Revision: d0a38203bf2f4c2a380f9efc7f33b4dc4858d63f

URL: https://github.com/llvm/llvm-project/commit/d0a38203bf2f4c2a380f9efc7f33b4dc4858d63f
DIFF: https://github.com/llvm/llvm-project/commit/d0a38203bf2f4c2a380f9efc7f33b4dc4858d63f.diff

LOG: [NFC] Remove assert from AArch64TargetLowering::LowerCTTZ. (#193474)

The assert is overly complex and pretty much boils down to "true". In
the unlikely event it would fire the lowering is perfectly safe anyway.

Added: 
    

Modified: 
    llvm/lib/Target/AArch64/AArch64ISelLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
index 38371b0f1dcd7..c5c8ce9505b7a 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -11753,12 +11753,6 @@ SDValue AArch64TargetLowering::LowerCTPOP_PARITY(SDValue Op,
 
 SDValue AArch64TargetLowering::LowerCTTZ(SDValue Op, SelectionDAG &DAG) const {
   EVT VT = Op.getValueType();
-  assert(VT.isScalableVector() || VT == MVT::v8i8 || VT == MVT::v16i8 ||
-         VT == MVT::v8i16 || VT == MVT::v4i16 || VT == MVT::v2i32 ||
-         VT == MVT::v4i32 ||
-         useSVEForFixedLengthVectorVT(
-             VT, /*OverrideNEON=*/Subtarget->useSVEForFixedLengthVectors()));
-
   SDLoc DL(Op);
   SDValue RBIT = DAG.getNode(ISD::BITREVERSE, DL, VT, Op.getOperand(0));
   return DAG.getNode(ISD::CTLZ, DL, VT, RBIT);


        


More information about the llvm-commits mailing list