vitalybuka added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:489
+ if (TC.getSanitizerArgs().needsScudoRt()) {
+ for (const auto &Lib : {"scudo", "scudo_cxx"}) {
----------------
We need this for both legacy and standalone scudo?
================
Comment at: compiler-rt/lib/scudo/scudo_platform.h:19
-#if !SANITIZER_LINUX && !SANITIZER_FUCHSIA
+#if !SANITIZER_LINUX && !SANITIZER_FUCHSIA && !SANITIZER_WINDOWS
# error "The Scudo hardened allocator is not supported on this platform."
----------------
For the reason @kcc mentioned, if possible could you not update legacy scudo at
at all, and keep only stuff needed for standalone version?
BTW. Patches to remove legacy one are welcomed.
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
