melver wrote:

Alright, I think this is as far as I can push it. This PR now contains 3 
commits just to keep the story clear (happy to squash them if that's better).

Besides the Linux kernel prototype integration, I took this version and tested 
it on our internal codebase that has used `-Wthread-safety` for years, so there 
is broad application and coverage of the feature in general -- with this latest 
version, I have found 1 real bug, and 1 very dubious pattern (see FIXME test), 
but otherwise no further issues.

But because there's a real chance this alias analysis might confuse existing 
users with new warnings, we should hide it behind `-Wthread-safety-beta`.

PTAL.

https://github.com/llvm/llvm-project/pull/142955
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to