russell.gallop added a comment. In D96120#2591368 <https://reviews.llvm.org/D96120#2591368>, @mstorsjo wrote:
> Yes, it would need something similar - I tried whipping something together, > which after some tweaks seems to work: > Feel free to squash that into your patch (which saves me a bit of effort) :-) Thanks for doing that. I've added this in. > ...I did test building the double-free.cpp test with `-fsanitize=scudo`, and > it seems to work as it should for all three cases it tests... Great, thanks. 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
