[PATCH] D42389: [ubsan] Add preinit initializer for ubsan

[Francis Ricci via Phabricator via llvm-commits]

fjricci added a comment.

> No, we need it to support configuring the runtime using setenv for the reasons described in the review of https://reviews.llvm.org/D39827.

Afaik, those reasons are android app-specific. We could theoretically still use getenv() on android, since preinit arrays aren't supported there anyway.

> I guess we don't need them here as we don't need them for any other sanitizers.

Are there any other sanitizers that we link in statically on android? That could be the difference?


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D42389