[llvm] 895ed64 - [AArch64] LowerCONCAT_VECTORS - merge getNumOperands() calls. NFCI.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 17 03:23:19 PDT 2021


Author: Simon Pilgrim
Date: 2021-08-17T11:23:03+01:00
New Revision: 895ed64009c024f9e6608f574f9ab083e421ae57

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

LOG: [AArch64] LowerCONCAT_VECTORS - merge getNumOperands() calls. NFCI.

Improves on the unused variable fix from rG4357562067003e25ab343a2d67a60bd89cd66dbf

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 9bbeb151d56b..883621973b07 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -10461,11 +10461,10 @@ SDValue AArch64TargetLowering::LowerCONCAT_VECTORS(SDValue Op,
 
   if (isTypeLegal(Op.getOperand(0).getValueType())) {
     unsigned NumOperands = Op->getNumOperands();
-    (void)NumOperands;
     assert(NumOperands > 1 && isPowerOf2_32(NumOperands) &&
            "Unexpected number of operands in CONCAT_VECTORS");
 
-    if (Op.getNumOperands() == 2)
+    if (NumOperands == 2)
       return Op;
 
     // Concat each pair of subvectors and pack into the lower half of the array.


        


More information about the llvm-commits mailing list