Changeset 44940ee for libcfa/src/concurrency/invoke.h

Timestamp:

Feb 25, 2026, 12:56:30 AM (27 hours ago)

Author:

Matthew Au-Yeung <mw2auyeu@…>

Branches:

stuck-waitfor-destruct

Children:

88bb0b4

Parents:

a0548c2

Message:

Revert "Add a generated hash to fix stuck waitfor comparing static inline mutex destructors"

This reverts commit a30fceb1a73c4ef2bbee39a2b5406da881f51111.

File:

• libcfa/src/concurrency/invoke.h (modified) (4 diffs)

libcfa/src/concurrency/invoke.h

@@ -159 +159 @@
                 // last function that acquired monitors
                 fptr_t func;
-
-                // hash-based function identity for cross-TU matching
-                func_id_t func_id;
         };
 
@@ -291 +288 @@
                         (this.size){0};
                         (this.func){NULL};
-                        (this.func_id){0};
-                }
-
-                static inline void ?{}(__monitor_group_t & this, struct monitor$ ** data, __lock_size_t size, fptr_t func, func_id_t func_id) {
+                }
+
+                static inline void ?{}(__monitor_group_t & this, struct monitor$ ** data, __lock_size_t size, fptr_t func) {
                         (this.data){data};
                         (this.size){size};
                         (this.func){func};
-                        (this.func_id){func_id};
                 }
 
@@ -304 +299 @@
                         if( (lhs.data != 0) != (rhs.data != 0) ) return false;
                         if( lhs.size != rhs.size ) return false;
-                        // Use hash-based comparison when both sides have a valid func_id,
-                        // otherwise fall back to function pointer comparison (for library code
-                        // like join() that cannot compute the hash at compile time).
-                        if( lhs.func_id != 0 && rhs.func_id != 0 ) {
-                                if( lhs.func_id != rhs.func_id ) return false;
-                        } else {
-                                if( lhs.func != rhs.func ) return false;
-                        }
+                        if( lhs.func != rhs.func ) return false;
 
                         // Check that all the monitors match
@@ -326 +314 @@
                         lhs.size = rhs.size;
                         lhs.func = rhs.func;
-                        lhs.func_id = rhs.func_id;
                 }
         }