Need option for coverage files directory

steleman / address-sanitizer

Automatically exported from code.google.com/p/address-sanitizer

0 stars 0 forks source link

Need option for coverage files directory #338

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

On Linux, we have a hack to chdir to a directory where we want coverage files. 
We need an option in ASAN_OPTIONS to specify path to directory where we want 
those files. This helps to remove the hack and also fixes other issues we need 
to be inside the application archive directory [so that e.g. pdb on windows get 
located, etc].

Original issue reported on code.google.com by infe...@chromium.org on 10 Sep 2014 at 3:41

GoogleCodeExporter commented 9 years ago

ASAN_OPTIONS=coverage_dir=/foo/bar

I'll update the docs shortly

Original comment by konstant...@gmail.com on 10 Sep 2014 at 3:50

GoogleCodeExporter commented 9 years ago

So, the option already exist in clang used by chromium atm or did you just 
added it to clang trunk ?

Original comment by infe...@chromium.org on 10 Sep 2014 at 4:01

GoogleCodeExporter commented 9 years ago

the flag was added in late May, so it is already in Chrome's clang.
http://llvm.org/viewvc/llvm-project?view=revision&revision=209815

Original comment by konstant...@gmail.com on 10 Sep 2014 at 4:04

GoogleCodeExporter commented 9 years ago

Apparently, coverage_dir is already documented at 
https://code.google.com/p/address-sanitizer/wiki/Flags
Done. :)

Original comment by konstant...@gmail.com on 10 Sep 2014 at 4:05

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Thanks!, Sorry i should have checked docs first.

Original comment by infe...@chromium.org on 11 Sep 2014 at 7:45