[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