[PATCH] D73951: [Clang] [Driver]Add logic to search for flang frontend

Richard Barton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 14 08:50:37 PST 2020


richard.barton.arm added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:268
+def fortran_fe : Separate<["-"], "fortran-fe">, InternalDriverOpt,
+  HelpText<"Name for native Fortran compiler">;
 
----------------
richard.barton.arm wrote:
> This is not right. I think the option points the driver at the name of the Fortran frontend to call. I don't think that has to be native or otherwise. Suggest changing this string to "Name for Fortran frontend"
I prefer the new option name though - thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73951





More information about the cfe-commits mailing list