================ @@ -1061,18 +1061,20 @@ def Most : DiagGroup<"most", [ ]>; // Thread Safety warnings -def ThreadSafetyAttributes : DiagGroup<"thread-safety-attributes">; -def ThreadSafetyAnalysis : DiagGroup<"thread-safety-analysis">; -def ThreadSafetyPrecise : DiagGroup<"thread-safety-precise">; -def ThreadSafetyReference : DiagGroup<"thread-safety-reference">; -def ThreadSafetyNegative : DiagGroup<"thread-safety-negative">; +def ThreadSafetyAttributes : DiagGroup<"thread-safety-attributes">; +def ThreadSafetyAnalysis : DiagGroup<"thread-safety-analysis">; +def ThreadSafetyPrecise : DiagGroup<"thread-safety-precise">; +def ThreadSafetyReference : DiagGroup<"thread-safety-reference">; +def ThreadSafetyReferenceReturn : DiagGroup<"thread-safety-reference-return">; ---------------- aeubanks wrote:
it's fine to put this in `ThreadSafety` and have it on by default (which it should be because it's improving thread safety analysis) since now people can turn it off with `-Wno-thread-safety-reference` while cleaning up their codebase https://github.com/llvm/llvm-project/pull/68572 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits