diff mbox

[ARM,testsuite] fix test c-c++-common/tm/omp.c

Message ID 000001ccd795$9a6d1200$cf473600$@Yorsh@arm.com
State New
Headers show

Commit Message

Greta Yorsh Jan. 20, 2012, 5:04 p.m. UTC 
PING!
http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01813.html


> -----Original Message-----
> From: Greta Yorsh [mailto:Greta.Yorsh@arm.com]
> Sent: 17 November 2011 10:36
> To: gcc-patches@gcc.gnu.org
> Cc: 'nickc@redhat.com'; Richard Earnshaw; 'paul@codesourcery.com';
> 'aldyh@redhat.com'
> Subject: [PATCH, ARM, testsuite] fix test c-c++-common/tm/omp.c
> 
> The testcase c-c++-common/tm/omp.c fails on arm-none-eabi:
> 
> Executing: arm-none-eabi-gcc /work/local-checkouts/gcc-
> fsf/gcc/testsuite/c-c++-common/tm/omp.c -fgnu-tm -fopenmp -S  -o omp.s
> arm-none-eabi-gcc: error: unrecognized command line option '-pthread'
> FAIL: c-c++-common/tm/omp.c
> 
> The attached patch adds the appropriate directive to the test:
> /* { dg-require-effective-target pthread } */
> 
> 
> -- Greta
> 
> 
> gcc/testsuite/ChangeLog
> 
> 2011-11-16  Greta Yorsh  <Greta.Yorsh@arm.com>
> 
> 	* c-c++-common/tm/omp.c: Require target with pthread support.

Comments

Aldy Hernandez Jan. 20, 2012, 7:51 p.m. UTC | #1 
On 01/20/12 11:04, Greta Yorsh wrote:
> PING!
> http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01813.html

This looks like an obviously correct patch, but I'll defer to the Arm 
maintainers.
Jakub Jelinek Jan. 20, 2012, 8:01 p.m. UTC | #2 
On Fri, Jan 20, 2012 at 05:04:41PM -0000, Greta Yorsh wrote:
> PING!
> http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01813.html

This is ok for trunk.  Thanks.

	Jakub
diff mbox

Patch

diff --git a/gcc/testsuite/c-c++-common/tm/omp.c b/gcc/testsuite/c-c++-common/tm/omp.c
index b9fcc76..b664a6f 100644
--- a/gcc/testsuite/c-c++-common/tm/omp.c
+++ b/gcc/testsuite/c-c++-common/tm/omp.c
@@ -1,5 +1,6 @@ 
 /* { dg-do compile } */
 /* { dg-options "-fgnu-tm -fopenmp" } */
+/* { dg-require-effective-target pthread } */
 
 __attribute__ ((transaction_pure))
 unsigned long rdtsc();