russell.gallop added a comment. In D86694#2242140 <https://reviews.llvm.org/D86694#2242140>, @cryptoad wrote:
> That's awesome! Is it meant to eventually be committed or only be used for > comparison purposes? I'd like it to be committed, but can't claim I know the code from https://reviews.llvm.org/D42519 well enough. The good news is that I can build LLVM on Windows with this. Is there a good sanity check that it is actually using Scudo rather than silently using the standard alloc? You marked D42519 <https://reviews.llvm.org/D42519> as WIP, can you remember what was still TBD? Also, it might make sense to separate out the "use" of sanitize=Scudo in LLVM, from providing Windows support. I put them together here for evaluation purposes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86694/new/ https://reviews.llvm.org/D86694 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits