[PATCH] D52395: Thread safety analysis: Require exclusive lock for passing by non-const reference

[Delesley Hutchins via Phabricator via cfe-commits]

delesley added a comment.

With respect to data, I really think these patches should be tested against Google's code base, because otherwise you're going to start seeing angry rollbacks.  However, I don't work on the C++ team any more, and I don't have time to do it.  When I was actively developing the analysis, I spent about 90% of my time just running tests and fixing the code base.  Each incremental improvement in the analysis itself was a hard upward slog.  If you're going to be adding lots of improvements, we need to have someone at Google running point.


Repository:
  rC Clang

https://reviews.llvm.org/D52395