[PATCH] D101281: [flang] Remove `%f18` from LIT configuration files
Andrzej Warzynski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 26 02:41:03 PDT 2021
awarzynski created this revision.
Herald added a reviewer: sscalpone.
awarzynski requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
`%f18` was originally introduced to represent the old Flang driver,
`f18`. With the introduction of the new driver, `flang-new`, we have
been switching to `%flang` (compiler driver) and `%flang_fc1` (frontend
driver) as more generic alternatives.
As most tests have been portend to use the new LIT variables instead of
`%f18`, this is good time to remove it from lit.cfg.py. There's only one
test left that requires the old driver to run. It's updated with:
! REQUIRES: old-flang-driver
This way we preserve its semantics while reducing the number of
variables in LIT configuration.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D101281
Files:
flang/test/Driver/help-f18.f90
flang/test/lit.cfg.py
Index: flang/test/lit.cfg.py
===================================================================
--- flang/test/lit.cfg.py
+++ flang/test/lit.cfg.py
@@ -42,6 +42,8 @@
# config.
if config.include_flang_new_driver_test:
config.available_features.add('new-flang-driver')
+else:
+ config.available_features.add('old-flang-driver')
# test_source_root: The root path where tests are located.
config.test_source_root = os.path.dirname(__file__)
@@ -62,11 +64,7 @@
# For each occurrence of a flang tool name, replace it with the full path to
# the build directory holding that tool.
-tools = [
- ToolSubst('%f18', command=FindTool('f18'),
- unresolved='fatal')
-]
-
+tools = []
if config.include_flang_new_driver_test:
tools.append(ToolSubst('%flang', command=FindTool('flang-new'), unresolved='fatal'))
tools.append(ToolSubst('%flang_fc1', command=FindTool('flang-new'),
Index: flang/test/Driver/help-f18.f90
===================================================================
--- flang/test/Driver/help-f18.f90
+++ flang/test/Driver/help-f18.f90
@@ -1,7 +1,9 @@
-! RUN: %f18 -h 2>&1 | FileCheck %s
-! RUN: %f18 -help 2>&1 | FileCheck %s
-! RUN: %f18 --help 2>&1 | FileCheck %s
-! RUN: %f18 -? 2>&1 | FileCheck %s
+! REQUIRES: old-flang-driver
+
+! RUN: %flang -h 2>&1 | FileCheck %s
+! RUN: %flang -help 2>&1 | FileCheck %s
+! RUN: %flang --help 2>&1 | FileCheck %s
+! RUN: %flang -? 2>&1 | FileCheck %s
! CHECK: f18: LLVM Fortran compiler
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101281.340464.patch
Type: text/x-patch
Size: 1466 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210426/22e0d555/attachment.bin>
More information about the llvm-commits
mailing list