[PATCH] D153131: [clang analysis][thread-safety] Handle return-by-reference...
Aaron Puchert via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Oct 7 06:43:50 PDT 2023
aaronpuchert added a comment.
In D153131#4653345 <https://reviews.llvm.org/D153131#4653345>, @aeubanks wrote:
> This is finding lots of real issues in code, which is awesome, but could I request that this be put under a separate warning flag so we can toggle off just the new functionality and turn it on as we clean our codebase? e.g. `-W[no-]thread-safety-analysis-return`
Fine for me, but we might want to remove it again after one or two releases. I'm not sure how to communicate that this is just a “transitory” flag.
And it should be included by default in `-Wthread-safety-reference`, so that users of that flag see the new warnings/errors, and can demote them to warnings while fixing them. To emphasize the subflag status, I'd suggest something like `-Wthread-safety-reference-return`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153131/new/
https://reviews.llvm.org/D153131
More information about the cfe-commits
mailing list