Hi!
On 2024-08-07T14:08:42+0200, Tobias Burnus <tbur...@baylibre.com> wrote:
> --- /dev/null
> +++ b/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-1.C
> +// { dg-additional-options -foffload-options=-fdump-tree-optimized { target
> { offload_device_nvptx || offload_target_amdgcn } } }
Note here: 'offload_device_nvptx' vs. 'offload_target_amdgcn', but...
> +// { dg-final { only_for_offload_target amdgcn-amdhsa
> scan-offload-tree-dump-not "omp_initial_device;" "optimized" { target
> offload_target_amdgcn } } }
> +// { dg-final { only_for_offload_target amdgcn-amdhsa scan-offload-tree-dump
> "v1\\._x = 5;" "optimized" { target offload_target_amdgcn } } }
> +// { dg-final { only_for_offload_target nvptx-none
> scan-offload-tree-dump-not "omp_initial_device;" "optimized" { target
> offload_target_nvptx } } }
> +// { dg-final { only_for_offload_target nvptx-none scan-offload-tree-dump
> "v1\\._x = 5;" "optimized" { target offload_target_nvptx } } }
... here: 'offload_target_nvptx', 'offload_target_amdgcn', resulting in a
few UNRESOLVEDs.
> [Etc.]
Pushed to trunk branch commit 9f5d22e3e2b8e4532896a4f3837cb86006d5930c
"OpenMP: Constructors and destructors for "declare target" static aggregates:
Fix effective-target keyword in test cases",
see attached.
Grüße
Thomas
>From 9f5d22e3e2b8e4532896a4f3837cb86006d5930c Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <tschwi...@baylibre.com>
Date: Fri, 9 Aug 2024 11:23:15 +0200
Subject: [PATCH] OpenMP: Constructors and destructors for "declare target"
static aggregates: Fix effective-target keyword in test cases
(Most of) the tests added in commit f1bfba3a9b3f31e3e06bfd1911c9f223869ea03f
"OpenMP: Constructors and destructors for "declare target" static aggregates"
had a mismatch between dump file production and its scanning; the former needs
to use 'offload_target_nvptx' (like 'offload_target_amdgcn'), not
'offload_device_nvptx'.
libgomp/
* testsuite/libgomp.c++/static-aggr-constructor-destructor-1.C:
Fix effective-target keyword.
* testsuite/libgomp.c++/static-aggr-constructor-destructor-2.C:
Likewise.
* testsuite/libgomp.c-c++-common/target-is-initial-host-2.c:
Likewise.
* testsuite/libgomp.c-c++-common/target-is-initial-host.c:
Likewise.
* testsuite/libgomp.fortran/target-is-initial-host-2.f90:
Likewise.
* testsuite/libgomp.fortran/target-is-initial-host.f: Likewise.
* testsuite/libgomp.fortran/target-is-initial-host.f90: Likewise.
---
.../libgomp.c++/static-aggr-constructor-destructor-1.C | 2 +-
.../libgomp.c++/static-aggr-constructor-destructor-2.C | 2 +-
.../testsuite/libgomp.c-c++-common/target-is-initial-host-2.c | 2 +-
libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host.c | 2 +-
libgomp/testsuite/libgomp.fortran/target-is-initial-host-2.f90 | 2 +-
libgomp/testsuite/libgomp.fortran/target-is-initial-host.f | 2 +-
libgomp/testsuite/libgomp.fortran/target-is-initial-host.f90 | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-1.C b/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-1.C
index b5aafc8cabc..a704e39411d 100644
--- a/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-1.C
+++ b/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-1.C
@@ -1,6 +1,6 @@
// { dg-do run }
// { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" }
-// { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } }
+// { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } }
// { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } }
// { dg-final { scan-tree-dump-times "_GLOBAL__off_I_v1" 1 "gimple" } }
diff --git a/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-2.C b/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-2.C
index 9652a721bbe..de481aadd34 100644
--- a/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-2.C
+++ b/libgomp/testsuite/libgomp.c++/static-aggr-constructor-destructor-2.C
@@ -1,6 +1,6 @@
// { dg-do run }
// { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" }
-// { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } }
+// { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } }
// { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } }
// { dg-final { scan-tree-dump-times "_GLOBAL__off_I_v1" 1 "gimple" } }
diff --git a/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host-2.c b/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host-2.c
index 313d188a871..eabc394c8cf 100644
--- a/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host-2.c
+++ b/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host-2.c
@@ -4,7 +4,7 @@
/* Check whether 'omp_is_initial_device()' is NOT compile-time optimized. */
/* { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" } */
-/* { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } } */
+/* { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } } */
/* { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } } */
diff --git a/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host.c b/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host.c
index 423727ce55f..3b1283a5979 100644
--- a/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host.c
+++ b/libgomp/testsuite/libgomp.c-c++-common/target-is-initial-host.c
@@ -3,7 +3,7 @@
/* Check whether 'omp_is_initial_device()' is properly compile-time optimized. */
/* { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" } */
-/* { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } } */
+/* { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } } */
/* { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } } */
diff --git a/libgomp/testsuite/libgomp.fortran/target-is-initial-host-2.f90 b/libgomp/testsuite/libgomp.fortran/target-is-initial-host-2.f90
index e06ced20645..56521044eb6 100644
--- a/libgomp/testsuite/libgomp.fortran/target-is-initial-host-2.f90
+++ b/libgomp/testsuite/libgomp.fortran/target-is-initial-host-2.f90
@@ -3,7 +3,7 @@
! Check whether 'omp_is_initial_device()' is NOT compile-time optimized. */
! { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" }
-! { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } }
+! { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } }
! { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } }
diff --git a/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f b/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f
index fec4a3f1c1c..34b327e3901 100644
--- a/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f
+++ b/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f
@@ -1,7 +1,7 @@
! Check whether 'omp_is_initial_device()' is properly compile-time optimized. */
! { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" }
-! { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } }
+! { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } }
! { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } }
diff --git a/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f90 b/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f90
index f8a645fc488..bed99635d42 100644
--- a/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f90
+++ b/libgomp/testsuite/libgomp.fortran/target-is-initial-host.f90
@@ -1,7 +1,7 @@
! Check whether 'omp_is_initial_device()' is properly compile-time optimized. */
! { dg-additional-options "-fdump-tree-gimple -fdump-tree-optimized" }
-! { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_device_nvptx || offload_target_amdgcn } } }
+! { dg-additional-options -foffload-options=-fdump-tree-optimized { target { offload_target_nvptx || offload_target_amdgcn } } }
! { dg-final { scan-tree-dump-times "omp_is_initial_device" 1 "gimple" } }
--
2.34.1
