[llvm] 0c4309b - IR: Avoid repeating assert condition in Value destructor (#136340)

via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 19 11:45:07 PDT 2025


Author: Matt Arsenault
Date: 2025-04-19T20:45:04+02:00
New Revision: 0c4309bcc4154562ae91c3705a9e6c398833119b

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

LOG: IR: Avoid repeating assert condition in Value destructor (#136340)

Added: 
    

Modified: 
    llvm/lib/IR/Value.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/IR/Value.cpp b/llvm/lib/IR/Value.cpp
index 6c52ced5f73b2..0eaf75b7bfaec 100644
--- a/llvm/lib/IR/Value.cpp
+++ b/llvm/lib/IR/Value.cpp
@@ -98,9 +98,10 @@ Value::~Value() {
     dbgs() << "While deleting: " << *VTy << " %" << getName() << "\n";
     for (auto *U : users())
       dbgs() << "Use still stuck around after Def is destroyed:" << *U << "\n";
+
+    llvm_unreachable("Uses remain when a value is destroyed!");
   }
 #endif
-  assert(materialized_use_empty() && "Uses remain when a value is destroyed!");
 
   // If this value is named, destroy the name.  This should not be in a symtab
   // at this point.


        


More information about the llvm-commits mailing list