diff options
author | Aaron Puchert <aaronpuchert@alice-dsl.net> | 2025-04-15 23:21:34 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-04-15 23:21:34 +0200 |
commit | 9c73eba8aa17cb7ca4248ab1c7f67ea7ec9b50b1 (patch) | |
tree | 9a73317d6a54d5d607a9d59e6e1990341534ac90 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | d30a5b41fe72a1dd83714d3e21fd539b91e63c8c (diff) | |
download | llvm-9c73eba8aa17cb7ca4248ab1c7f67ea7ec9b50b1.zip llvm-9c73eba8aa17cb7ca4248ab1c7f67ea7ec9b50b1.tar.gz llvm-9c73eba8aa17cb7ca4248ab1c7f67ea7ec9b50b1.tar.bz2 |
Merge similar Clang Thread Safety attributes (#135561)
Some of the old lock-based and new capability-based spellings behave
basically in the same way, so merging them simplifies the code
significantly.
There are two minor functional changes: we only warn (instead of an
error) when the try_acquire_capability attribute is used on something
else than a function. The alternative would have been to produce an
error for the old spelling, but we seem to only warn for all function
attributes, so this is arguably more consistent.
The second change is that we also check the first argument (which is the
value returned for a successful try-acquire) for `this`. But from what I
can tell, this code is defunct anyway at the moment (see #31414).
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions