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

Sourabh Singh Tomar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 11 09:56:36 PDT 2021


SouraVX added a comment.

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 :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103177



More information about the llvm-commits mailing list