-
Notifications
You must be signed in to change notification settings - Fork 176
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
Error stop improvements #473
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. LGTM.
I originally authored this routine, which is updated for supported compilers. The logic selecting it in CMake was no longer relevant.
I think there is some value in keeping the pre-Fortran 2018 error stop workaround. The current default branch still compiles with Intel 19, with this patch we would drop support for all Intel versions before 2021.3.
Also, while Intel 2021.1 compiles stdlib, the support for Fortran 2018 error stop is buggy and will always report exit status 128 even if zero is passed as exit code. |
@scivision can you explain in a sentence or two what this PR changes or improves? I understand the part about the more robust test driver. What is the tradeoff for dropping support for earlier Intel compilers? |
I originally authored this error_stop routine. The logic selecting it in CMake was no
longer relevant. I made a driver for the error stop test to make it more reliable.
oneAPI 2021.1 on MacOS has bugs breaking stdlib_error, so I update CI to latest oneAPI for MacOS.