[PATCH] D96120: [scudo] Port scudo sanitizer to Windows

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 14:46:33 PST 2021


hctim added a comment.

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

> In D96120#2597877 <https://reviews.llvm.org/D96120#2597877>, @kcc wrote:
>
>> I am afraid we will have to delete the older (non-standalone) variant entirely. 
>> (And the sooner the better)
>
> Okay, think we found a few things out by doing this before porting standalone.
>
> If I find time I’ll have a go at removing the sanitizer version.
>
> The scudo documentation (https://llvm.org/docs/ScudoHardenedAllocator.html) still seems to refer to the sanitizer version so will need updating.

When we remove the non-standalone variant, it would be great to move `-fsanitize=scudo` over to scudo-standalone :).


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

https://reviews.llvm.org/D96120



More information about the llvm-commits mailing list