sourceryinstitute / OpenCoarrays

A parallel application binary interface for Fortran 2018 compilers.
http://www.opencoarrays.org
BSD 3-Clause "New" or "Revised" License
247 stars 56 forks source link

Defect: Test cases use pre-F2018 co_reduce()-parameter operator, which is renamed in F2018 to operation #745

Closed vehre closed 2 years ago

vehre commented 2 years ago

In gcc>=12 the Fortran Standard 2018's co_reduce(... operation=...) is implement. Code using pre-2018 Fortran (and still using operator=...) will stop compiling.

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information.

System information including:

To help us debug your issue please explain:

What you were trying to do (and why)

What happened (include command output, screenshots, logs, etc.)

What you expected to happen

Step-by-step reproduction instructions to reproduce the error/bug

Compile testsuite with gcc>=12.

vehre commented 2 years ago

Fix is available at: https://github.com/sourceryinstitute/OpenCoarrays/tree/issue-745-rename-co_reduce-operator-for-F2018

The fix uses the pre-processor to use the different identifiers depending on the gcc-version used. Therefore the files had to renamed to *.F90, i.e. with a capital F which by convention enables preprocessor-processing.

GitHub
GitHub - sourceryinstitute/OpenCoarrays at issue-745-rename-co_reduce-operator-for-F2018
A parallel application binary interface for Fortran 2018 compilers. - GitHub - sourceryinstitute/OpenCoarrays at issue-745-rename-co_reduce-operator-for-F2018
rouson commented 2 years ago

Thanks, @vehre! Please open a pull request to merge this fix into the main branch.

@everythingfunctional please let me know if you can review and merge the PR

everythingfunctional commented 2 years ago

Sounds straightforward enough. Open a PR and I'll review it.

vehre commented 2 years ago

The pull reQuest is at: https://github.com/sourceryinstitute/OpenCoarrays/pull/746

rouson commented 2 years ago

PR #746 has been merged so I'm closing this issue.