Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding NAG Fortran compiler (nagfor) support in the CMake build system. #686

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

ACSimon33
Copy link
Contributor

@ACSimon33 ACSimon33 commented Jun 29, 2022

Description

The changes in this pull request add the necessary compiler flags for the NAG Fortran compiler in the CMake files. The following flags were added in places where they are necessary:

  • -recursive (available since nagfor 7.0)
  • -i8 (64bit integers)
  • -ieee=full (no FPE traps)
  • -dcfuns (non-standard double precision complex intrinsic functions: CDABS, DCMPLX, DCONJG, DIMAG, DREAL)
  • -thread_safe (enable thread safety)
  • -quiet (Suppress compiler banner and summary)

Disabled warnings:

  • -w=obs (obsolescent features)
  • -w=x77 (F77 extensions)
  • -w=ques (questionable usage)
  • -w=unused (unused variables, imports, procedures, etc.)

Miscellaneous changes:

The nagfor compiler detected a few lines in the test files that had line lengths of 73. I refactored those to less or equal 72 characters.

Checklist

  • The documentation has been updated. (not applicable)
  • If the PR solves a specific issue, it is set to be closed on merge. (not applicable)

@langou langou merged commit 2789c76 into Reference-LAPACK:master Jun 30, 2022
@ACSimon33 ACSimon33 deleted the cmake_nagfor_support branch July 1, 2022 10:25
@ACSimon33 ACSimon33 mentioned this pull request Jul 1, 2022
2 tasks
@julielangou julielangou added this to the LAPACK 3.11.0 milestone Nov 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants