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

Francis Ricci via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 16:11:15 PST 2018


fjricci added inline comments.


================
Comment at: lib/ubsan/ubsan_flags.cc:29
 
+const char *GetFlag(const char *flag) {
+  // We cannot call getenv() from inside a preinit array initializer
----------------
vitalybuka wrote:
> static here and above?
`MaybeCallUbsanDefaultOptions` is declared extern, but I'll make `GetFlag` static.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D42389





More information about the llvm-commits mailing list