[PATCH] D153001: [clang][ThreadSafety] Add __builtin_instance_member (WIP)
    Timm Bäder via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Jun 28 01:03:11 PDT 2023
    
    
  
tbaeder added a comment.
Didn't remember and re-checked, using it that way makes the implementation harder I think:
  tsa2.c:7:54: error: incomplete definition of type 'struct Mutex'
      7 |   int counter GUARDED_BY(__builtin_instance_member(M)->M);
        |                                                      ^
  tsa2.c:6:10: note: forward declaration of 'struct Mutex'
      6 |   struct Mutex *M;
        |          ^
  1 error generated.
and I didn't immediately know how to fix that.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153001/new/
https://reviews.llvm.org/D153001
    
    
More information about the cfe-commits
mailing list