[PATCH] D73951: [Clang] [Driver]Add logic to search for flang frontend

[Andrzej Warzynski via Phabricator via cfe-commits]

awarzynski added a comment.

The suggested flag, `-fortran-fe`, feels quite powerful and I wonder whether it's required this early into the development of the Flang driver?

IIUC, this flag would be used to workaround the fact that currently `flang` is a bash script and hence can't be used when the driver is in the `flang` mode. `-fortran-fe` could be use to redirect the call to `flang-tmp` (the temporary flang driver that will eventually replace the bash script with similar name). Wouldn't it be less work if we just used `flang-tmp` instead of `flang` here: https://github.com/llvm/llvm-project/blob/42a56bf63f699a620a57c34474510d9937ebf715/clang/lib/Driver/ToolChains/Flang.cpp#L72? Once the driver is ready to replace the bash script, we could revert the change in Flang.cpp (`flang-tmp` -> `flang`).

Are there any other usecases that we should cater for at this point? Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73951