[PATCH] D13892: Apply modernize-use-default to compiler-rt.

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 20 05:48:14 PDT 2015


bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.

lg with comments.


================
Comment at: lib/tsan/rtl/tsan_mutex.cc:130
@@ -129,3 @@
-InternalDeadlockDetector::InternalDeadlockDetector() {
-  // Rely on zero initialization because some mutexes can be locked before ctor.
-}
----------------
Please keep the comments.

================
Comment at: lib/tsan/rtl/tsan_report.cc:71
@@ -70,3 @@
-ReportDesc::~ReportDesc() {
-  // FIXME(dvyukov): it must be leaking a lot of memory.
-}
----------------
dito


http://reviews.llvm.org/D13892





More information about the cfe-commits mailing list