[PATCH] [tsan] print error summary line

Dmitry Vyukov dvyukov at google.com
Wed Feb 6 06:13:11 PST 2013


  other than that LGTM


================
Comment at: tsan/rtl/tsan_report.cc:171
@@ +170,3 @@
+static ReportStack *SkipTsanInternalFrame(ReportStack *ent) {
+  if (ent && ent->next && internal_strstr(ent->file, "tsan/rtl/"))
+    return ent->next;
----------------
It's not necessary "tsan/rtl", some interceptors are in sanitizer_common.
Please unify it with logic in tsan_rtl_report.cc.



http://llvm-reviews.chandlerc.com/D377



More information about the llvm-commits mailing list