Page MenuHomePhabricator

[flang] Extended the flang driver options to include gfortran equivalents to pgf90 specific options.
ClosedPublic

Authored by AlexisPerry on Jul 13 2020, 9:39 AM.

Details

Summary

Added gfortran equivalents of pgf90's -Mfixed, -Mfree, -Mextend, -Mstandard, -Munlimited and also added -fdefault-double-8

Diff Detail

Event Timeline

AlexisPerry created this revision.Jul 13 2020, 9:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 13 2020, 9:39 AM
AlexisPerry added a project: Restricted Project.Jul 13 2020, 9:39 AM
sscalpone accepted this revision.Jul 13 2020, 10:11 AM
This revision is now accepted and ready to land.Jul 13 2020, 10:11 AM

This LGTM and was reviewed already under D83488. Happy to approve.

Correction for handling of OpenMP enabling flags

Looks like you needed to squash this into the preivous revision of this patch before submitting? I only see the -fopenmp change now.

Timezones are not helping here - I need to clock off for the day. I think revision 1 + revision 2 are correct when combined.

I am happy for you to push the combination of revision 1 and revision 2. I'll approve it on that basis.

Extended the flang driver options to include gfortran equivalents to pgf90 specific options.

  • Added gfortran equivalents of pgf90's -Mfixed, -Mfree, -Mextend, -Mstandard, -Munlimited
  • Also added -fdefault-double-8
  • Added in a switch to pass through the correct OpenMP-enabling flag depending on if F18_FC=pgf90 or =gfortran
This revision was automatically updated to reflect the committed changes.