[PATCH] D63607: [clang][driver] Add basic --driver-mode=flang support for fortran

Wed Oct 30 05:46:32 PDT 2019

thakis added a comment.

Sorry, I had missed the RfC, but it looks like there wasn't a lot of discussion on it anyways.

Adding fortran support to clang's driver has been suggested and decided against before, see "[cfe-commits] [RFC and PATCH] Fortran"

In D63607#1726895 <https://reviews.llvm.org/D63607#1726895>, @teemperor wrote:

> Could we have the `clang/test/Driver/flang/flang.F90` and `clang/test/Driver/flang/flang.f90` files in different directories please? As macOS's FS is case-insensitive, those two files have the same path from macOS perspective which is causing a whole bunch of issues (including breaking git even with 'core.ignorecase').

One issue is that the test flakily fails about half the time

http://45.33.8.238/mac/2498/summary.html fail
http://45.33.8.238/mac/2499/summary.html pass
http://45.33.8.238/mac/2500/summary.html fail
http://45.33.8.238/mac/2501/summary.html pass
http://45.33.8.238/mac/2502/summary.html fail
http://45.33.8.238/mac/2503/summary.html pass
http://45.33.8.238/mac/2504/summary.html fail

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63607/new/

https://reviews.llvm.org/D63607