[llvm] d1ef449 - [AArch64] Fold one-use variables into assert

Benjamin Kramer via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 17 10:43:51 PDT 2020


Author: Benjamin Kramer
Date: 2020-04-17T19:43:06+02:00
New Revision: d1ef44982f36b8783119fa18d6ef9edbd9617027

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

LOG: [AArch64] Fold one-use variables into assert

Avoids unused variable warnings in Release builds.

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 a52dd21d44b1..a0c17a9f0704 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -3150,9 +3150,7 @@ SDValue AArch64TargetLowering::LowerINTRINSIC_WO_CHAIN(SDValue Op,
     if (!Ty.isVector())
       report_fatal_error("Unexpected type for aarch64_neon_vsli");
 
-    uint64_t ShiftAmount = Op.getConstantOperandVal(3);
-    unsigned ElemSizeInBits = Ty.getScalarSizeInBits();
-    assert(ShiftAmount <= ElemSizeInBits);
+    assert(Op.getConstantOperandVal(3) <= Ty.getScalarSizeInBits());
 
     bool IsShiftRight = IntNo == Intrinsic::aarch64_neon_vsri;
     unsigned Opcode = IsShiftRight ? AArch64ISD::VSRI : AArch64ISD::VSLI;


        


More information about the llvm-commits mailing list