[PATCH] D28029: Increase the treshold in unit test to accomodate for qurantine size increase.

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 21 13:57:28 PST 2016


alekseyshl updated this revision to Diff 82269.
alekseyshl added a comment.

- Add an explanatory comment.


https://reviews.llvm.org/D28029

Files:
  lib/asan/tests/asan_noinst_test.cc


Index: lib/asan/tests/asan_noinst_test.cc
===================================================================
--- lib/asan/tests/asan_noinst_test.cc
+++ lib/asan/tests/asan_noinst_test.cc
@@ -176,8 +176,9 @@
     pthread_t t;
     PTHREAD_CREATE(&t, NULL, ThreadedQuarantineTestWorker, 0);
     PTHREAD_JOIN(t, 0);
-    size_t mmaped2 = __sanitizer_get_heap_size();
-    EXPECT_LT(mmaped2 - mmaped1, 320U * (1 << 20));
+    size_t mmaped2 = __sanitizer_get_heap_size();i
+    // TODO: Figure out why this much memory is required.
+    EXPECT_LT(mmaped2 - mmaped1, 352U * (1 << 20));
   }
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28029.82269.patch
Type: text/x-patch
Size: 593 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161221/27a7e8db/attachment.bin>


More information about the llvm-commits mailing list