[PATCH] D7203: [asan] Set abort_on_error=1 by default on OS X
Filipe Cabecinhas
filcab+llvm.phabricator at gmail.com
Tue Jul 21 18:11:38 PDT 2015
filcab added a comment.
That was done because on some platforms (like the PS4. Possibly (some)
Windows people would like it too) symbolize_vs_style=true is the default.
This build configuration wouldn't allow anyone to test the sanitizer
libraries *and* ship the binary they tested with.
Making sure that the platform's defaults are ok is a matter of creating a
test which only runs on that platform and matches whatever defaults are
different from "the usual defaults".
Having the tests depend on the user remembering to set the proper
ASAN_OPTIONS for testing is much more fragile and prone to problems (and a
hidden dependency on the testing environment). And it would still require
all tests that set ASAN_OPTIONS to something also set it to
ASAN_OPTIONS=$ASAN_OPTIONS:... anyway.
Filipe
http://reviews.llvm.org/D7203
More information about the llvm-commits
mailing list