[PATCH] D96120: [scudo] Port scudo sanitizer to Windows
Mitch Phillips via Phabricator via cfe-commits
cfe-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 cfe-commits
mailing list