[PATCH] D86694: [scudo] Allow -fsanitize=scudo on Linux and Windows (WIP, don't land as is)

[Kostya Kortchinsky via Phabricator via llvm-commits]

cryptoad added a comment.

In D86694#2268182 <https://reviews.llvm.org/D86694#2268182>, @russell.gallop wrote:

> 



> Thanks. I'm looking at porting the standalone variant, drawing on D42519 <https://reviews.llvm.org/D42519> and Windows support from sanitizer_common. Does that sound like a reasonable approach?

Yup, the structure is a tad different, and other platforms (Fuchsia, Linux cpp) have some implementation of the platform specific functions.

> Would these be good flags to use:
>
>   set SCUDO_OPTIONS=allocator_release_to_os_interval_ms=-1:QuarantineSizeKb=0:ThreadLocalQuarantineSizeKb=0:DeleteSizeMismatch=0:DeallocationTypeMismatch=0

Yup, no release, no quarantine is a great. Size class map compile time options could also make a difference.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86694/new/

https://reviews.llvm.org/D86694