[PATCH] D125788: [flang][driver] Rename `flang-new` as `flang`
Andrzej Warzynski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 3 00:49:25 PDT 2023
awarzynski added a subscriber: everythingfunctional.
awarzynski added a comment.
In D125788#4556324 <https://reviews.llvm.org/D125788#4556324>, @MaskRay wrote:
> If you continue this work, consider landing this rename in multiple phases, e.g.
>
> - Use lit substitutions instead of `flang-new` and other preparatory stuff so that the next step has very little change
> - change flang-new to flang and add flang-new as a symlink for compatibility
> - remove the symlink `flang-new`, when it is no longer used.
>
> Many build bots can fail in weird ways and ensuring that we will not revert a large change prevents causing churn to the tests.
Thank you, these are great pointers! CC @everythingfunctional
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125788/new/
https://reviews.llvm.org/D125788
More information about the cfe-commits
mailing list