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

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 13:10:33 PST 2018


pcc added a comment.

> @pcc If we revert getenv() part, we would not be able to use getenv/setenv for sanitizer options. but I guess we don't need them here as we don't need them for any other sanitizers.

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


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D42389





More information about the llvm-commits mailing list