[compiler-rt] r294056 - [scudo] Fix buildbot test error on ARM

Kostya Kortchinsky via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 13:59:01 PST 2017


Author: cryptoad
Date: Fri Feb  3 15:59:00 2017
New Revision: 294056

URL: http://llvm.org/viewvc/llvm-project?rev=294056&view=rev
Log:
[scudo] Fix buildbot test error on ARM

Summary:
The assumption __sanitizer_get_heap_size() == 0 (introduced in D29341) at the
start of a program appears to be incorrect on some ARM machines
(SizeClassAllocator32).

This should fix the test while I investigate the issue.

Reviewers: kcc, alekseyshl

Reviewed By: alekseyshl

Subscribers: aemerson, rengolin, llvm-commits

Differential Revision: https://reviews.llvm.org/D29516

Modified:
    compiler-rt/trunk/test/scudo/interface.cpp

Modified: compiler-rt/trunk/test/scudo/interface.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/scudo/interface.cpp?rev=294056&r1=294055&r2=294056&view=diff
==============================================================================
--- compiler-rt/trunk/test/scudo/interface.cpp (original)
+++ compiler-rt/trunk/test/scudo/interface.cpp Fri Feb  3 15:59:00 2017
@@ -39,8 +39,8 @@ int main(int argc, char **argv)
   }
   if (!strcmp(argv[1], "heap-size")) {
     // Ensures that __sanitizer_get_heap_size can be called before any other
-    // allocator function. At this point, this heap size should be 0.
-    assert(__sanitizer_get_heap_size() == 0);
+    // allocator function.
+    assert(__sanitizer_get_heap_size() >= 0);
   }
 
   return 0;




More information about the llvm-commits mailing list