[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test case #168830

kiranktp · 2025-11-20T06:22:03Z

Replace use of flang -fc1 with %flang_fc1 in few test case

llvmbot · 2025-11-20T06:22:34Z

@llvm/pr-subscribers-flang-semantics
@llvm/pr-subscribers-flang-fir-hlfir

@llvm/pr-subscribers-flang-openmp

Author: Kiran Kumar T P (kiranktp)

Changes

Replace use of flang -fc1 with %flang_fc1 in few test case

Full diff: https://github.com/llvm/llvm-project/pull/168830.diff

14 Files Affected:

(modified) flang/test/Lower/Intrinsics/command_argument_count.f90 (+1-1)
(modified) flang/test/Lower/Intrinsics/modulo.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/Todo/omp-clause-indirect.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/Todo/omp-declarative-allocate.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/Todo/omp-declare-reduction-initsub.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/Todo/omp-declare-reduction.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/Todo/omp-declare-simd.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/cray-pointers01.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/cray-pointers02.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/parallel-lastprivate-clause-scalar.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/parallel-wsloop-reduction-byref.f90 (+1-1)
(modified) flang/test/Lower/OpenMP/parallel-wsloop-reduction.f90 (+1-1)
(modified) flang/test/Semantics/indirect01.f90 (+1-1)
(modified) flang/test/Semantics/indirect02.f90 (+2-2)

diff --git a/flang/test/Lower/Intrinsics/command_argument_count.f90 b/flang/test/Lower/Intrinsics/command_argument_count.f90
index a30b27d664fc0..35737e8aee1cf 100644
--- a/flang/test/Lower/Intrinsics/command_argument_count.f90
+++ b/flang/test/Lower/Intrinsics/command_argument_count.f90
@@ -1,6 +1,6 @@
 ! RUN: bbc -emit-fir %s -o - | FileCheck %s
 ! bbc doesn't have a way to set the default kinds so we use flang driver
-! RUN: flang -fc1 -fdefault-integer-8 -emit-fir %s -o - | FileCheck --check-prefixes=CHECK,CHECK-64  %s
+! RUN: %flang_fc1 -fdefault-integer-8 -emit-fir %s -o - | FileCheck --check-prefixes=CHECK,CHECK-64  %s
 
 ! CHECK-LABEL: argument_count_test
 subroutine argument_count_test()
diff --git a/flang/test/Lower/Intrinsics/modulo.f90 b/flang/test/Lower/Intrinsics/modulo.f90
index 460000a923447..f31a8092233bf 100644
--- a/flang/test/Lower/Intrinsics/modulo.f90
+++ b/flang/test/Lower/Intrinsics/modulo.f90
@@ -1,5 +1,5 @@
 ! RUN: bbc -emit-fir -hlfir=false %s -o - | FileCheck %s -check-prefixes=HONORINF,ALL
-! RUN: flang -fc1 -menable-no-infs -emit-fir -flang-deprecated-no-hlfir %s -o - | FileCheck %s -check-prefixes=CHECK,ALL,%if flang-supports-f128-math %{F128%} %else %{F64%}
+! RUN: %flang_fc1 -menable-no-infs -emit-fir -flang-deprecated-no-hlfir %s -o - | FileCheck %s -check-prefixes=CHECK,ALL,%if flang-supports-f128-math %{F128%} %else %{F64%}
 
 ! ALL-LABEL: func @_QPmodulo_testr(
 ! ALL-SAME: %[[arg0:.*]]: !fir.ref<f64>{{.*}}, %[[arg1:.*]]: !fir.ref<f64>{{.*}}, %[[arg2:.*]]: !fir.ref<f64>{{.*}}) {
diff --git a/flang/test/Lower/OpenMP/Todo/omp-clause-indirect.f90 b/flang/test/Lower/OpenMP/Todo/omp-clause-indirect.f90
index d441cac47f5da..82efa8818a83c 100644
--- a/flang/test/Lower/OpenMP/Todo/omp-clause-indirect.f90
+++ b/flang/test/Lower/OpenMP/Todo/omp-clause-indirect.f90
@@ -1,6 +1,6 @@
 ! This test checks the lowering of OpenMP Indirect Clause when used with the Declare Target directive
 
-! RUN: not flang -fc1 -emit-fir -fopenmp -fopenmp-version=52 %s 2>&1 | FileCheck %s
+! RUN: not %flang_fc1 -emit-fir -fopenmp -fopenmp-version=52 %s 2>&1 | FileCheck %s
 
 module functions
   implicit none
diff --git a/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate.f90 b/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate.f90
index 3307eb2505b71..7cae8051fda77 100644
--- a/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate.f90
+++ b/flang/test/Lower/OpenMP/Todo/omp-declarative-allocate.f90
@@ -1,6 +1,6 @@
 ! This test checks lowering of OpenMP allocate Directive.
 
-! RUN: not flang -fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
+! RUN: not %flang_fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
 
 program main
   integer :: x, y
diff --git a/flang/test/Lower/OpenMP/Todo/omp-declare-reduction-initsub.f90 b/flang/test/Lower/OpenMP/Todo/omp-declare-reduction-initsub.f90
index 30630465490b2..f2c744599fdc5 100644
--- a/flang/test/Lower/OpenMP/Todo/omp-declare-reduction-initsub.f90
+++ b/flang/test/Lower/OpenMP/Todo/omp-declare-reduction-initsub.f90
@@ -1,7 +1,7 @@
 ! This test checks lowering of OpenMP declare reduction Directive, with initialization
 ! via a subroutine. This functionality is currently not implemented.
 
-! RUN: not flang -fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
+! RUN: not %flang_fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
 
 !CHECK: not yet implemented: OpenMPDeclareReductionConstruct
 subroutine initme(x,n)
diff --git a/flang/test/Lower/OpenMP/Todo/omp-declare-reduction.f90 b/flang/test/Lower/OpenMP/Todo/omp-declare-reduction.f90
index db50c9ac8ee9d..11e83cc710f05 100644
--- a/flang/test/Lower/OpenMP/Todo/omp-declare-reduction.f90
+++ b/flang/test/Lower/OpenMP/Todo/omp-declare-reduction.f90
@@ -1,6 +1,6 @@
 ! This test checks lowering of OpenMP declare reduction Directive.
 
-! RUN: not flang -fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
+! RUN: not %flang_fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
 
 subroutine declare_red()
   integer :: my_var
diff --git a/flang/test/Lower/OpenMP/Todo/omp-declare-simd.f90 b/flang/test/Lower/OpenMP/Todo/omp-declare-simd.f90
index be1ac2db5dfa4..a63bfb284642c 100644
--- a/flang/test/Lower/OpenMP/Todo/omp-declare-simd.f90
+++ b/flang/test/Lower/OpenMP/Todo/omp-declare-simd.f90
@@ -1,6 +1,6 @@
 ! This test checks lowering of OpenMP declare simd Directive.
 
-// RUN: not flang -fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
+// RUN: not %flang_fc1 -emit-fir -fopenmp %s 2>&1 | FileCheck %s
 
 subroutine sub(x, y)
   real, intent(inout) :: x, y
diff --git a/flang/test/Lower/OpenMP/cray-pointers01.f90 b/flang/test/Lower/OpenMP/cray-pointers01.f90
index d3a5a3cdd39a3..01c6b8bbce260 100644
--- a/flang/test/Lower/OpenMP/cray-pointers01.f90
+++ b/flang/test/Lower/OpenMP/cray-pointers01.f90
@@ -1,5 +1,5 @@
 ! Test lowering of Cray pointee references.
-! RUN: flang -fc1 -emit-hlfir -fopenmp %s -o - 2>&1 | FileCheck %s
+! RUN: %flang_fc1 -emit-hlfir -fopenmp %s -o - 2>&1 | FileCheck %s
 
 module test_host_assoc_cray_pointer
   ! CHECK-LABEL: fir.global @_QMtest_host_assoc_cray_pointerEivar : i64
diff --git a/flang/test/Lower/OpenMP/cray-pointers02.f90 b/flang/test/Lower/OpenMP/cray-pointers02.f90
index 79d838702e4b0..4dd1c9e6215b8 100644
--- a/flang/test/Lower/OpenMP/cray-pointers02.f90
+++ b/flang/test/Lower/OpenMP/cray-pointers02.f90
@@ -1,5 +1,5 @@
 ! Test lowering of Cray pointee references.
-! RUN: flang -fc1 -emit-hlfir -fopenmp %s -o - 2>&1 | FileCheck %s
+! RUN: %flang_fc1 -emit-hlfir -fopenmp %s -o - 2>&1 | FileCheck %s
 
 ! CHECK-LABEL: func.func @_QQmain() attributes {fir.bindc_name = "TEST_CRAY_POINTERS_02"}
 program test_cray_pointers_02
diff --git a/flang/test/Lower/OpenMP/parallel-lastprivate-clause-scalar.f90 b/flang/test/Lower/OpenMP/parallel-lastprivate-clause-scalar.f90
index 50fd8e11089ea..efab520e96a5d 100644
--- a/flang/test/Lower/OpenMP/parallel-lastprivate-clause-scalar.f90
+++ b/flang/test/Lower/OpenMP/parallel-lastprivate-clause-scalar.f90
@@ -1,7 +1,7 @@
 ! This test checks lowering of `LASTPRIVATE` clause for scalar types.
 
 ! RUN: bbc -fopenmp -emit-hlfir %s -o - | FileCheck %s
-! RUN: flang -fc1 -fopenmp -emit-hlfir %s -o - | FileCheck %s
+! RUN: %flang_fc1 -fopenmp -emit-hlfir %s -o - | FileCheck %s
 
 !CHECK: func @_QPlastprivate_character(%[[ARG1:.*]]: !fir.boxchar<1>{{.*}}) {
 !CHECK-DAG: %[[ARG1_UNBOX:.*]]:2 = fir.unboxchar
diff --git a/flang/test/Lower/OpenMP/parallel-wsloop-reduction-byref.f90 b/flang/test/Lower/OpenMP/parallel-wsloop-reduction-byref.f90
index 3f44f292cb6a0..23d387a2ef839 100644
--- a/flang/test/Lower/OpenMP/parallel-wsloop-reduction-byref.f90
+++ b/flang/test/Lower/OpenMP/parallel-wsloop-reduction-byref.f90
@@ -1,7 +1,7 @@
 ! Check that for parallel do, reduction is only processed for the loop
 
 ! RUN: bbc -fopenmp --force-byref-reduction -emit-hlfir %s -o - | FileCheck %s
-! RUN: flang -fc1 -fopenmp -mmlir --force-byref-reduction -emit-hlfir %s -o - | FileCheck %s
+! RUN: %flang_fc1 -fopenmp -mmlir --force-byref-reduction -emit-hlfir %s -o - | FileCheck %s
 
 ! CHECK: omp.parallel {
 ! CHECK: omp.wsloop private({{.*}}) reduction(byref @add_reduction_byref_i32
diff --git a/flang/test/Lower/OpenMP/parallel-wsloop-reduction.f90 b/flang/test/Lower/OpenMP/parallel-wsloop-reduction.f90
index a206eef52da5a..225f8297db65b 100644
--- a/flang/test/Lower/OpenMP/parallel-wsloop-reduction.f90
+++ b/flang/test/Lower/OpenMP/parallel-wsloop-reduction.f90
@@ -1,7 +1,7 @@
 ! Check that for parallel do, reduction is only processed for the loop
 
 ! RUN: bbc -fopenmp -emit-hlfir %s -o - | FileCheck %s
-! RUN: flang -fc1 -fopenmp -emit-hlfir %s -o - | FileCheck %s
+! RUN: %flang_fc1 -fopenmp -emit-hlfir %s -o - | FileCheck %s
 
 ! CHECK: omp.parallel {
 ! CHECK: omp.wsloop private({{.*}}) reduction(@add_reduction_i32
diff --git a/flang/test/Semantics/indirect01.f90 b/flang/test/Semantics/indirect01.f90
index 59850662275d9..81fcfbc94aa3e 100644
--- a/flang/test/Semantics/indirect01.f90
+++ b/flang/test/Semantics/indirect01.f90
@@ -1,6 +1,6 @@
 ! This test checks the lowering of OpenMP Indirect Clause when used with the Declare Target directive
 
-! RUN: not flang -fopenmp -fopenmp-version=52 %s 2>&1 | FileCheck %s
+! RUN: not %flang -fopenmp -fopenmp-version=52 %s 2>&1 | FileCheck %s
 
 module functions
   implicit none
diff --git a/flang/test/Semantics/indirect02.f90 b/flang/test/Semantics/indirect02.f90
index 273f8856626b7..cfae406486c67 100644
--- a/flang/test/Semantics/indirect02.f90
+++ b/flang/test/Semantics/indirect02.f90
@@ -1,7 +1,7 @@
 ! This test checks the lowering of OpenMP Indirect Clause when used with the Declare Target directive
 
-! RUN: not flang -fopenmp -fopenmp-version=50 %s 2>&1 | FileCheck %s --check-prefix="CHECK-50"
-! RUN: not flang -fopenmp -fopenmp-version=52 %s 2>&1 | FileCheck %s --check-prefix="CHECK-52"
+! RUN: not %flang -fopenmp -fopenmp-version=50 %s 2>&1 | FileCheck %s --check-prefix="CHECK-50"
+! RUN: not %flang -fopenmp -fopenmp-version=52 %s 2>&1 | FileCheck %s --check-prefix="CHECK-52"
 
 module functions
   implicit none

Thirumalai-Shaktivel

LGTM, thank you!

github-actions · 2025-11-20T06:38:27Z

🐧 Linux x64 Test Results

4060 tests passed
202 tests skipped

NimishMishra

Thanks. LGTM

…se (llvm#168830) Replace use of flang -fc1 with %flang_fc1 in few test case

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test case

bffb273

Replace use of flang -fc1 with %flang_fc1 in few test case

llvmbot added flang Flang issues not falling into any other category flang:fir-hlfir flang:openmp flang:semantics labels Nov 20, 2025

kiranktp requested review from NimishMishra, Thirumalai-Shaktivel, kaviya2510, mjklemm and tblah November 20, 2025 06:27

Thirumalai-Shaktivel approved these changes Nov 20, 2025

View reviewed changes

Merge branch 'main' into NFC_test_case_modification

95431ce

NimishMishra approved these changes Nov 20, 2025

View reviewed changes

kiranktp merged commit dc343d2 into llvm:main Nov 20, 2025
10 checks passed

aadeshps-mcw pushed a commit to aadeshps-mcw/llvm-project that referenced this pull request Nov 26, 2025

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test ca…

7b9c452

…se (llvm#168830) Replace use of flang -fc1 with %flang_fc1 in few test case

Priyanshu3820 pushed a commit to Priyanshu3820/llvm-project that referenced this pull request Nov 26, 2025

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test ca…

180b381

…se (llvm#168830) Replace use of flang -fc1 with %flang_fc1 in few test case

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test case #168830

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test case #168830

Uh oh!

kiranktp commented Nov 20, 2025

Uh oh!

llvmbot commented Nov 20, 2025 •

edited

Loading

Uh oh!

Thirumalai-Shaktivel left a comment

Uh oh!

github-actions bot commented Nov 20, 2025

Uh oh!

NimishMishra left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test case #168830

[NFC][flang] Replace use of flang -fc1 with %flang_fc1 in few test case #168830

Uh oh!

Conversation

kiranktp commented Nov 20, 2025

Uh oh!

llvmbot commented Nov 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Thirumalai-Shaktivel left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Nov 20, 2025

🐧 Linux x64 Test Results

Uh oh!

NimishMishra left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

llvmbot commented Nov 20, 2025 •

edited

Loading