[PATCH] D27375: Let ubsan search UBSAN_SYMBOLIZER_PATH for llvm-symbolizer

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 16:33:04 PST 2016


In that case, this change does the UBSAN_SYMBOLIZER_PATH part :-D

On Fri, Dec 2, 2016 at 7:28 PM, Evgeniy Stepanov via Phabricator <
reviews at reviews.llvm.org> wrote:

> eugenis added a comment.
>
> I can imagine a separate environment variable being more convenient in
> some cases. I don't think we should deprecate it. Also, printing stuff can
> break user programs, so it should not be done by default, and doing it only
> with verbosity=1 is almost useless, no one will see it.
>
> I'd add LLVM_SYMBOLIZER_PATH, maybe add UBSAN_SYMBOLIZER_PATH (just for
> consistency), and change the docs to point to LLVM_SYMBOLIZER_PATH and
> external_symbolizer_path.
>
>
> https://reviews.llvm.org/D27375
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161202/313cddf3/attachment.html>


More information about the llvm-commits mailing list