[llvm] f140491 - [LLVM][CppCompile] Fix unused variable warning after 0e025418 (#163203)

via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 13 08:00:46 PDT 2025


Author: Sohaib Iftikhar
Date: 2025-10-13T15:00:42Z
New Revision: f140491dcc906e18aa67a6c204774245c69fc606

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

LOG: [LLVM][CppCompile] Fix unused variable warning after 0e025418 (#163203)

[LLVM][CppCompile] Fix unused variable warning after 0e025418

Fixes unused variable warning for libTransform with non debug builds.
Folds the unused variable into the assert.

Added: 
    

Modified: 
    llvm/lib/Transforms/Utils/SSAUpdaterBulk.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Utils/SSAUpdaterBulk.cpp b/llvm/lib/Transforms/Utils/SSAUpdaterBulk.cpp
index ccb86eb2cb0fc..fb39fddde72e2 100644
--- a/llvm/lib/Transforms/Utils/SSAUpdaterBulk.cpp
+++ b/llvm/lib/Transforms/Utils/SSAUpdaterBulk.cpp
@@ -269,8 +269,7 @@ static bool replaceIfIdentical(PHINode &PHI, PHINode &ReplPHI) {
 
 bool EliminateNewDuplicatePHINodes(BasicBlock *BB,
                                    BasicBlock::phi_iterator FirstExistingPN) {
-  auto NewPHIs = make_range(BB->phis().begin(), FirstExistingPN);
-  assert(!PHIAreRefEachOther(NewPHIs));
+  assert(!PHIAreRefEachOther(make_range(BB->phis().begin(), FirstExistingPN)));
 
   // Deduplicate new PHIs first to reduce the number of comparisons on the
   // following new -> existing pass.


        


More information about the llvm-commits mailing list