[PATCH] D54469: Introduce new `disable_init` ASan option that is only supported on platforms where `SANITIZER_SUPPORTS_DISABLED_INIT` is true. Currently this is only supported on Darwin.

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 29 08:54:46 PST 2018


delcypher added inline comments.


================
Comment at: lib/asan/asan_internal.h:115
+// For sanitizer shared library dlopen() support
+bool InitIsViaDlopen();
+void HandleDlopenInit();
----------------
kcc wrote:
> I wonder if you can turn these two functions into one. 
Sure it's possible. I'll have a go at doing this and will update the patch shortly.


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D54469





More information about the llvm-commits mailing list