Re: [PATCH] D12021: Remove test cases, which rely on the default sanitizer blacklists.

2015-08-13 Thread Peter Collingbourne via cfe-commits
pcc accepted this revision. pcc added a reviewer: pcc. pcc added a comment. This revision is now accepted and ready to land. LGTM Yes, this can be a separate driver test. http://reviews.llvm.org/D12021 ___ cfe-commits mailing list cfe-commits@lists

Re: [PATCH] D12021: Remove test cases, which rely on the default sanitizer blacklists.

2015-08-13 Thread Ivan Krasin via cfe-commits
krasin added a comment. In http://reviews.llvm.org/D12021#224102, @pcc wrote: > You could create a fake resource directory which contains only the > `asan_blacklist.txt` file. See `test/Driver/Inputs/resource_dir` for an > example of this kind of thing. I believe it's unnecessary. The default

Re: [PATCH] D12021: Remove test cases, which rely on the default sanitizer blacklists.

2015-08-13 Thread Peter Collingbourne via cfe-commits
pcc added a comment. You could create a fake resource directory which contains only the `asan_blacklist.txt` file. See `test/Driver/Inputs/resource_dir` for an example of this kind of thing. http://reviews.llvm.org/D12021 ___ cfe-commits mailing l