[PATCH] D103177: [flang][driver] Extend the `flang` bash script to act as a driver

Fri Jun 11 11:15:43 PDT 2021

klausler added a comment.

In D103177#2813671 <https://reviews.llvm.org/D103177#2813671>, @SouraVX wrote:

> Overall I support & I'd like to have one driver i.e `flang-new` considering all the effort that has gone in it and the long term plan associated with the driver(that will cater to the entire `flang` project as a primary interface). :)
>
> I can observe/note following benefits of having one driver:
>
> - Keeping things consistent/converge on `flang-new` will help in synchronizing the effort of the community. This should help anyone who wants to contribute, not to get intimidated by presence of 2 drivers(and implied effort of keeping the `feature parity` across both).
> - `flang-new` just now gained the `feature parity`  with existing driver `f18` (Thanks to @awarzynski and other folks involved :)), from now onwards either one has to bear the cost of maintenance & **NOT** breaking each other while progressing.
> - Speaking of the dependency on `clang`, I think this will remain and if people don't appreciate it - then this will require entire re-write, non-trivial effort, and needs a solid motivation to move forward along this path. **NOT** to mention, this entire activity may end up eating most (if not all) efforts that were planned to get the driver functional/usable to a point :)
>
> On the path forward, If people have interest in having `f18` (which I can see from above conversation), then at least we can mutually agree upon some clear timelines(2-3 weeks ?) as to when `f18` is Okay to remove completely. 
> This should help in synchronizing folks from both sides :)

That's fine with me; as I've said, I don't need to have f18.cpp in your tree to be able to work on my parts of the project.  Hypothetical future Fortran tool-writers who just want a parser & maybe semantics can reconstruct it when/if needed, I suppose.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103177