[flang-commits] [flang] [flang][OpenMP] Fix `omp-declarative-allocate-align.f90` expectations (PR #122675)

Kareem Ergawy via flang-commits flang-commits at lists.llvm.org
Mon Jan 13 00:56:01 PST 2025


https://github.com/ergawy created https://github.com/llvm/llvm-project/pull/122675

The test was effectively a no-op since we used `//` instead of `!` for `RUN` and `CHECK` lines. Also, we have to specify the proper OpenMP version.

>From 89f6827448e1369f7e083e98bf02b6f568afedea Mon Sep 17 00:00:00 2001
From: ergawy <kareem.ergawy at amd.com>
Date: Mon, 13 Jan 2025 02:53:33 -0600
Subject: [PATCH] [flang][OpenMP] Fix `omp-declarative-allocate-align.f90`
 expectations

The test was effectively a no-op since we used `//` instead of `!` for
`RUN` and `CHECK` lines. Also, we have to specify the proper OpenMP
version.
---
 .../test/Lower/OpenMP/Todo/omp-declarative-allocate-align.f90 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate-align.f90 b/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate-align.f90
index d0ed0cbb4c831d..8daf20e1ae400a 100644
--- a/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate-align.f90
+++ b/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate-align.f90
@@ -1,10 +1,10 @@
 ! This test checks lowering of OpenMP allocate Directive with align clause.
 
-// RUN: not flang -fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
+! RUN: not %flang_fc1 -emit-fir -fopenmp -fopenmp-version=51 %s 2>&1 | FileCheck %s
 
 program main
   integer :: x
 
-  // CHECK: not yet implemented: OpenMPDeclarativeAllocate
+  ! CHECK: not yet implemented: OpenMPDeclarativeAllocate
   !$omp allocate(x) align(32)
 end



More information about the flang-commits mailing list