[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