diff mbox

libsanitizer merge from upstream r196090

Message ID 52F0FCC6.5000300@partner.samsung.com
State New
Headers show

Commit Message

max Feb. 4, 2014, 2:44 p.m. UTC 
Hi,

 > FAIL: g++.dg/tsan/default_options.C  -O2  execution test
 >
 > Both these tests don't report anything (well, default_options.C
 > prints DONE),
 > simply succeed (with dg-shouldfail that is a failure)

I've finally fixed  g+.dg/tsan/default_options.C test (the check
dg-shouldfail should be removed).

OK to commit?

-Maxim

Comments

Jakub Jelinek Feb. 4, 2014, 2:54 p.m. UTC | #1 
On Tue, Feb 04, 2014 at 06:44:22PM +0400, Maxim Ostapenko wrote:
> Hi,
> 
> > FAIL: g++.dg/tsan/default_options.C  -O2  execution test
> >
> > Both these tests don't report anything (well, default_options.C
> > prints DONE),
> > simply succeed (with dg-shouldfail that is a failure)
> 
> I've finally fixed  g+.dg/tsan/default_options.C test (the check
> dg-shouldfail should be removed).
> 
> OK to commit?

Sure.

> 2014-02-04  Max Ostapenko  <m.ostapenko@partner.samsung.com>
> 
> 	* g++.dg/tsan/default_options.C: Invert check.
> 
> diff --git a/gcc/testsuite/g++.dg/tsan/default_options.C b/gcc/testsuite/g++.dg/tsan/default_options.C
> index f0c0ece..13e1984 100644
> --- a/gcc/testsuite/g++.dg/tsan/default_options.C
> +++ b/gcc/testsuite/g++.dg/tsan/default_options.C
> @@ -1,5 +1,3 @@
> -/* { dg-shouldfail "tsan" } */
> -
>  #include <pthread.h>
>  #include <stdio.h>

	Jakub
max Feb. 4, 2014, 3:04 p.m. UTC | #2 
Commited in 207472.

-Maxim
diff mbox

Patch

2014-02-04  Max Ostapenko  <m.ostapenko@partner.samsung.com>

	* g++.dg/tsan/default_options.C: Invert check.

diff --git a/gcc/testsuite/g++.dg/tsan/default_options.C b/gcc/testsuite/g++.dg/tsan/default_options.C
index f0c0ece..13e1984 100644
--- a/gcc/testsuite/g++.dg/tsan/default_options.C
+++ b/gcc/testsuite/g++.dg/tsan/default_options.C
@@ -1,5 +1,3 @@ 
-/* { dg-shouldfail "tsan" } */
-
 #include <pthread.h>
 #include <stdio.h>