aaron.ballman closed this revision.
aaron.ballman added a comment.
Committed the fix in r329930.
https://reviews.llvm.org/D44155
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
I think this code is ready to go in -- accepting my own revision so that I can
close. If @delesley spots any issues, they can be address post commit.
https://reviews.llvm.org/D4
aaron.ballman added a comment.
Ping.
https://reviews.llvm.org/D44155
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman updated this revision to Diff 138900.
aaron.ballman added a comment.
Now: with more context!
https://reviews.llvm.org/D44155
Files:
lib/Analysis/ThreadSafety.cpp
test/SemaCXX/warn-thread-safety-analysis.cpp
Index: test/SemaCXX/warn-thread-safety-analysis.cpp
aaron.ballman added a comment.
In https://reviews.llvm.org/D44155#1041584, @lebedev.ri wrote:
> Please upload patches with full context (`-U9`) :)
Ugh, yes. TortoiseSVN has no option for this, so I perpetually forget.
https://reviews.llvm.org/D44155
___
lebedev.ri added a comment.
Please upload patches with full context (`-U9`) :)
https://reviews.llvm.org/D44155
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman added a comment.
Ping.
https://reviews.llvm.org/D44155
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
