[PATCH] D46546: add suffix to report file name

Bill Torpey via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 22 03:50:52 PDT 2018


WallStProg added a comment.

In https://reviews.llvm.org/D46546#1150695, @filcab wrote:

> Check out `test/sanitizer_common/TestCases/options-include.cc` as an example you can use to adapt yours.
>  You might only need to change `env_asan_opts` to `env_tool_opts` to get it to work.
>
> Instead of having a positive `FIXME`, please change it to a negative: `windows and android don't support X` (which is what the line below is doing).


Sorry, but I have no idea what all those decorations are, nor do I have any desire to learn.  I simply copied and pasted from an existing test program to get something that worked.

Hopefully this will make into trunk at some point -- it's quite useful, and I'd love to be able to stop patching my local builds.


https://reviews.llvm.org/D46546





More information about the llvm-commits mailing list