awarzynski added a comment.

In D125788#3519403 <https://reviews.llvm.org/D125788#3519403>, @DavidSpickett 
wrote:

> Sounds good to me. https://reviews.llvm.org/D125796 for the bot side, let me 
> know if/when the first half of the change goes in.

https://reviews.llvm.org/D125832 for "patch 1".

In D125788#3521487 <https://reviews.llvm.org/D125788#3521487>, 
@kiranchandramohan wrote:

> It will be a good idea to,
> -> wait for the author of the original script and the throwaway driver, 
> @sscalpone.

👍🏻

> -> clarify that execution is still restricted to developers via a flag.

👍🏻  Nothing changes with regard to the compiler flags (there are no functional 
changes here, just the renaming).

> My personal preference for the renaming is `flang-to-external-fc` since it is 
> not specific to gfortran.

Makes sense, I can rename it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125788

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to