[PATCH] D83488: [flang] Change the default F18_FC to gfortran

[Richard Barton via Phabricator via llvm-commits]

richard.barton.arm requested changes to this revision.
richard.barton.arm added a comment.
This revision now requires changes to proceed.

Request one change to OpenMP handling, but the rest looks fine.



================
Comment at: flang/tools/f18/f18.cpp:636
   if (options.features.IsEnabled(Fortran::common::LanguageFeature::OpenMP)) {
-    driver.pgf90Args.push_back("-mp");
+    driver.F18_FCArgs.push_back("-mp");
   }
----------------
This needs to be amended to support gfortran. We'd need a similar switch on `isPGF90` to pass `-fopenmp` here. I think it might work better to be moved into the next if/else block so it can be set to gfortran/pgfortran there. 

Might also be better to add this as part of D83687 rather than this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83488