shithub: libvpx

Download patch

ref: 297c110dcb4c026b5d420e31f4f839b9b583e910
parent: 2d0c11093e9854980a84d9415da2eb6a4cd42647
parent: d7ba519b9f081d779aa225f3bd16ed6b7d782e87
author: Jerome Jiang <jianj@google.com>
date: Wed Aug 30 21:52:42 EDT 2017

Merge "Revert "Re-enable disabled tests under TSan.""

--- a/test/datarate_test.cc
+++ b/test/datarate_test.cc
@@ -258,6 +258,14 @@
   }
 }
 
+// Disabled for tsan, see:
+// https://bugs.chromium.org/p/webm/issues/detail?id=1049
+#if defined(__has_feature)
+#if __has_feature(thread_sanitizer)
+#define BUILDING_WITH_TSAN
+#endif
+#endif
+#ifndef BUILDING_WITH_TSAN
 TEST_P(DatarateTestLarge, DropFramesMultiThreads) {
   denoiser_on_ = 0;
   cfg_.rc_buf_initial_sz = 500;
@@ -277,6 +285,7 @@
   ASSERT_LE(cfg_.rc_target_bitrate, file_datarate_ * 1.4)
       << " The datarate for the file missed the target!";
 }
+#endif  // !BUILDING_WITH_TSAN
 
 class DatarateTestRealTime : public DatarateTestLarge {
  public:
@@ -393,6 +402,10 @@
   }
 }
 
+// Disabled for tsan, see:
+// https://bugs.chromium.org/p/webm/issues/detail?id=1049
+
+#ifndef BUILDING_WITH_TSAN
 TEST_P(DatarateTestRealTime, DropFramesMultiThreads) {
   denoiser_on_ = 0;
   cfg_.rc_buf_initial_sz = 500;
@@ -413,6 +426,7 @@
   ASSERT_LE(cfg_.rc_target_bitrate, file_datarate_ * 1.4)
       << " The datarate for the file missed the target!";
 }
+#endif
 
 TEST_P(DatarateTestRealTime, GFBoost) {
   denoiser_on_ = 0;