[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
Tue Nov 13 15:22:53 PST 2018


delcypher added a comment.

In https://reviews.llvm.org/D54469#1296906, @kubamracek wrote:

> Can this be triggered by a separate environment variable instead? That way we don't have to expose it in flags at all.


Sure. My original patch actually did this with a `SanitizeInitIsDisabled()` function which was platform specific. I thought moving it into flags was a bit cleaner and it also fixed a problem because some of the sanitizer's code doesn't behave that well if you skip parsing the flags.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D54469





More information about the llvm-commits mailing list