[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