[PATCH] D40038: [scudo] Soft and hard RSS limit checks

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 14 15:11:26 PST 2017


alekseyshl accepted this revision.
alekseyshl added a comment.
This revision is now accepted and ready to land.

Ok, add a TODO with the required improvements and let's get it in.



================
Comment at: test/scudo/rss.c:45
+    printf("Some of the malloc calls returned NULL (%d)\n", returned_null);
+  printf("Done\n");
+  return 0;
----------------
cryptoad wrote:
> alekseyshl wrote:
> > What's this for?
> I added it because it seems that if there is nothing on stdout or stderr, FileCheck complains and fails, which is the case for the "nolimit" tests.
There's --allow-empty flag, but this way is fine too.


https://reviews.llvm.org/D40038





More information about the llvm-commits mailing list