[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