[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