[PATCH] D42822: [sanitizer] Implement GetRSS on Windows
Kostya Kortchinsky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 9 11:35:41 PST 2018
cryptoad added a comment.
In https://reviews.llvm.org/D42822#1003689, @vitalybuka wrote:
> They fail if I break GetRSS on Linux.
> I didn't check if anything else is missing on windows to support soft_rss_limit_mb=
Currently Windows doesn't have a background thread that is needed for the RSS check (launched by MaybeStartBackgroundThread).
GetRSS here is implemented for Scudo support. The larger current WIP CL is at https://reviews.llvm.org/D42519, which includes the RSS test support (but it needs a lot of other stuff to land).
Scudo uses a per deallocation check as opposed to a background thread to not have to start a new thread.
Repository:
rCRT Compiler Runtime
https://reviews.llvm.org/D42822
More information about the llvm-commits
mailing list