[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