[PATCH] D39827: ubsan: Allow programs to use setenv to configure ubsan_standalone.

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

In https://reviews.llvm.org/D39827#920047, @vitalybuka wrote:

> we have interceptors for nicer support of deadly signals, so probably we'd like to install them ASAP


I'm not sure if that is too important, especially since a user has the choice of using (e.g.) ubsan+asan if they want early interception of signals.

> also there is weak __ubsan_default_options() which can be used instead of setenv

That won't always work. I'm thinking about the Android app use case where you may have multiple ubsan-instrumented libraries and a main program written in another language that wants to configure the runtime.

> it was added here https://reviews.llvm.org/rL213983

I found that change, but it does not explain why ubsan would want .preinit_array specifically instead of .init_array (aside from consistency with other sanitizers).


https://reviews.llvm.org/D39827