This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFC ThreadSanitizer tests
- From: Jakub Jelinek <jakub at redhat dot com>
- To: max <m dot ostapenko at partner dot samsung dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Yury Gribov <y dot gribov at samsung dot com>, Slava Garbuzov <v dot garbuzov at samsung dot com>
- Date: Thu, 5 Dec 2013 09:07:02 +0100
- Subject: Re: RFC ThreadSanitizer tests
- Authentication-results: sourceware.org; auth=none
- References: <52A01954 dot 60901 at partner dot samsung dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Dec 05, 2013 at 10:12:36AM +0400, max wrote:
> Hello,
>
> Here is a patch with initial ThreadSanitizer testsuite. It basically
> adds several tests from upstream LLVM testsuite.
> It works fine on x86_64 with patch from
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59188 applied.
>
> Ok to commit or should we wait for fix for 59188?
How costly the tests are? I see that the tests have at most 3 threads,
so it hopefully doesn't affect that much parallel testing, but how long
does it take to run make check-gcc check-g++ RUNTESTFLAGS=tsan.exp ?
How much memory does it need?
> 2013-12-05 Max Ostapenko <m.ostapenko@partner.samsung.com>
One extra space missing before <.
>
> * c-c++-common/tsan: New folder with tests added.
Instead of mentioning the directory in the ChangeLog, mention
the individual test files.
* c-c++-common/tsan/atomic_stack.c: New test.
...
> * lib/tsan-dg.exp: New testfiles.
: New file.
> * gcc.dg/tsan/tsan.exp: New testfiles.
: New file.
> * g++.dg/dg.exp: Add tsan directory to the list
> of folders that are handled specially.
* g++.dg/dg.exp: Prune tsan subdirectory.
Jakub