[PATCH] D97207: [flang][test] Share all driver test dirs between `f18` and `flang-new`
Tim Keith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 22 11:30:29 PST 2021
tskeith added a comment.
It looks like there are other tests in Flang-Driver that can be run on f18 as well. That doesn't have to happen in this change if you prefer not to.
================
Comment at: flang/test/Frontend/prescanner-diag.f90:5
+! RUN: %flang -E -I %S/Inputs/ %s 2>&1 | FileCheck %s
+! RUN: %flang_fc1 -E -I %S/Inputs/ %s 2>&1 | FileCheck %s
----------------
I think you need to keep the `%f18` command so that the test is run on f18 when the new driver is enabled. Unfortunately that means when it is disabled the same test is run on f18 three times, but none of these should take much time.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97207/new/
https://reviews.llvm.org/D97207
More information about the llvm-commits
mailing list