Hi,

On Tue, 7 Jul 2020 at 20:47, Nathan Sidwell <nat...@acm.org> wrote:
>
> I discovered I'd missed applying a testcase when fixing up the EOF token
> location a while back.
>
>      gcc/testsuite/
>              * c-c++-common/cpp/pragma-eof.c: New
>

The new testcase fails on targets that do not support openmp (I saw
problems on arm-eabi and aarch64-elf using newlib, where -lpthread was
failing).

I'm pushing the attached patch as obvious.

Thanks

Christophe

> --
> Nathan Sidwell
From 2f06fb713845dd4777f8b659cbe008a2a131cd80 Mon Sep 17 00:00:00 2001
From: Christophe Lyon <christophe.l...@linaro.org>
Date: Fri, 10 Jul 2020 12:42:47 +0000
Subject: [PATCH 1/1] testsuite: Fix c-c++-common/cpp/pragma-eof.c

Add missing fopenmp effective-target.

2020-07-10  Christophe Lyon  <christophe.l...@linaro.org>

gcc/testsuite/ChangeLog:
	* c-c++-common/cpp/pragma-eof.c: Add missing fopenmp
	effective-target.
---
 gcc/testsuite/c-c++-common/cpp/pragma-eof.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gcc/testsuite/c-c++-common/cpp/pragma-eof.c b/gcc/testsuite/c-c++-common/cpp/pragma-eof.c
index c72be80..c109435 100644
--- a/gcc/testsuite/c-c++-common/cpp/pragma-eof.c
+++ b/gcc/testsuite/c-c++-common/cpp/pragma-eof.c
@@ -1,6 +1,7 @@
 /* { dg-additional-options -fopenmp }  */
+/* { dg-require-effective-target fopenmp } */
 
-/* { dg-error "expected" "" { target *-*-* } 6 } */
+/* { dg-error "expected" "" { target *-*-* } 7 } */
 /* Make sure we see pragma_eol even though lacking new line.  *
 /* no newline at end of file.  */
 #pragma omp parallel
\ No newline at end of file
-- 
2.7.4

Reply via email to