[llvm] 1808618 - [unroll] Remove two dead variable assignments [nfc]

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 23 09:12:27 PST 2021


Author: Philip Reames
Date: 2021-11-23T09:12:20-08:00
New Revision: 18086186ab5ac7698db0fbc45c58cff2a7747da6

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

LOG: [unroll] Remove two dead variable assignments [nfc]

These variables are not out-params, and we immediately return after assigning them.  Thus, the assignments are dead and just confusing.

I believe these used to be out-params, but they're not any more.

Added: 
    

Modified: 
    llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp b/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
index 0b8b47fc932b..3d067d4d78eb 100644
--- a/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
+++ b/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
@@ -982,8 +982,6 @@ bool llvm::computeUnrollCount(
   if (UnrollFactor) {
     UP.Count = *UnrollFactor;
     UseUpperBound = (FullUnrollMaxTripCount == FullUnrollTripCount);
-    TripCount = FullUnrollTripCount;
-    TripMultiple = UP.UpperBound ? 1 : TripMultiple;
     return ExplicitUnroll;
   } else {
     UP.Count = FullUnrollTripCount;


        


More information about the llvm-commits mailing list