[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