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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D96120: [scudo] Por... Russell Gallop via Phabricator via cfe-commits
    • [PATCH] D96120: [scudo... Mitch Phillips via Phabricator via cfe-commits

Reply via email to