[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