[llvm] 9a3e8b1 - extractConstantWithoutWrapping - use const APInt& returned by SCEVConstant::getAPInt()

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 10 02:25:09 PDT 2020


Author: Simon Pilgrim
Date: 2020-07-10T10:24:29+01:00
New Revision: 9a3e8b11a8317b1a3d7440b0585b011cc9527494

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

LOG: extractConstantWithoutWrapping - use const APInt& returned by SCEVConstant::getAPInt()

Avoids unnecessary APInt copies and silences clang tidy warning.

Added: 
    

Modified: 
    llvm/lib/Analysis/ScalarEvolution.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Analysis/ScalarEvolution.cpp b/llvm/lib/Analysis/ScalarEvolution.cpp
index 75926aa3a960..48c686b73260 100644
--- a/llvm/lib/Analysis/ScalarEvolution.cpp
+++ b/llvm/lib/Analysis/ScalarEvolution.cpp
@@ -1353,7 +1353,7 @@ bool ScalarEvolution::proveNoWrapByVaryingStart(const SCEV *Start,
 static APInt extractConstantWithoutWrapping(ScalarEvolution &SE,
                                             const SCEVConstant *ConstantTerm,
                                             const SCEVAddExpr *WholeAddExpr) {
-  const APInt C = ConstantTerm->getAPInt();
+  const APInt &C = ConstantTerm->getAPInt();
   const unsigned BitWidth = C.getBitWidth();
   // Find number of trailing zeros of (x + y + ...) w/o the C first:
   uint32_t TZ = BitWidth;


        


More information about the llvm-commits mailing list