[PATCH] D87420: scudo: Introduce a new mechanism to let Scudo access a platform-specific TLS slot

[Kostya Kortchinsky via Phabricator via llvm-commits]

cryptoad added inline comments.


================
Comment at: compiler-rt/lib/scudo/standalone/tsd_shared.h:12
 
 #include "linux.h" // for getAndroidTlsPtr()
 #include "tsd.h"
----------------
Is this still necessary?


================
Comment at: compiler-rt/lib/scudo/standalone/tsd_shared.h:16
+#if SCUDO_HAS_PLATFORM_TLS_SLOT
+#include "scudo_platform_tls_slot.h"
+#endif
----------------
Is there some requirement as to where this should live, or build time include path requirement that should be documented?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87420/new/

https://reviews.llvm.org/D87420