[llvm-branch-commits] [llvm] IR: Remove reference counts from ConstantData (PR #137314)

Matt Arsenault via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue May 6 01:34:24 PDT 2025


================
@@ -518,17 +509,8 @@ class Value {
 
   /// This method should only be used by the Use class.
   void addUse(Use &U) {
-    if (hasUseList())
-      U.addToList(Uses.List);
-    else
-      U.addToList(Uses.Count);
-  }
-
-  void removeUse(Use &U) {
-    if (hasUseList())
-      U.removeFromList(Uses.List);
-    else
-      U.removeFromList(Uses.Count);
+    if (UseList || hasUseList())
----------------
arsenm wrote:

hasUseList is sufficient, not sure if this is really an optimization. I was going to remove this after to see if it makes a difference 

https://github.com/llvm/llvm-project/pull/137314


More information about the llvm-branch-commits mailing list