[compiler-rt] 3847986 - Fix the test case from D88686
Jianzhou Zhao via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 2 16:00:24 PDT 2020
Author: Jianzhou Zhao
Date: 2020-10-02T22:59:55Z
New Revision: 3847986fd2c838026b9d883bb61f2e419988c1a5
URL: https://github.com/llvm/llvm-project/commit/3847986fd2c838026b9d883bb61f2e419988c1a5
DIFF: https://github.com/llvm/llvm-project/commit/3847986fd2c838026b9d883bb61f2e419988c1a5.diff
LOG: Fix the test case from D88686
It seems that one buildnot RSS value is much higher after munmap than
local run.
Added:
Modified:
compiler-rt/test/dfsan/munmap_release_shadow.c
Removed:
################################################################################
diff --git a/compiler-rt/test/dfsan/munmap_release_shadow.c b/compiler-rt/test/dfsan/munmap_release_shadow.c
index 085844dfa692..98147dc695b9 100644
--- a/compiler-rt/test/dfsan/munmap_release_shadow.c
+++ b/compiler-rt/test/dfsan/munmap_release_shadow.c
@@ -41,7 +41,7 @@ int main(int argc, char **argv) {
assert(after_mmap >= before + mmap_cost_kb);
// OS does not release memory to the same level as the start of the program.
// The assert checks the memory after munmap up to a delta.
- const size_t delta = 5000;
- assert(after_munmap + mmap_cost_kb <= after_mmap + delta);
+ const size_t delta = 50000;
+ assert(after_munmap + delta <= after_mmap);
return 0;
}
More information about the llvm-commits
mailing list