[PATCH] [ASan] Add process basename to log name and error message to simplify analysis of sanitized systems logs.
Kostya Serebryany
kcc at google.com
Fri Feb 20 08:42:26 PST 2015
> > You mean initialize it in _asan_init, _msan_init, etc.? Ok.
>
Yes, create something like InitializeBinaryName() and GetBinaryName(), GetWhateverElseName() in sanitizer_common and call it in tool init routines.
> > I could but is there a good reason not to make this default (besides legacy scripts)?
>
legacy scripts is a good reason by itself, but more importantly, you are the first to request this in 4 years,
which means that most users don't need it. If they don't need it, extra bloat in the report will be a distraction.
If we see that the feature is useful we can always set the flag on by default.
REPOSITORY
rL LLVM
http://reviews.llvm.org/D7333
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list