-
Notifications
You must be signed in to change notification settings - Fork 2
Remove INTEGER*4 #266
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
Remove INTEGER*4 #266
Conversation
|
I wonder if it would be more portable using |
|
Yes, there are much more correct things to do that this. Or look at removing the MATMUL_ESSL branch altogether? I don't think it is used (but then is this whole file even used?). I don't particularly want to go down a rabbit hole at this point. |
|
Heh, I followed up with the next warning from Fortitude and it promptly complained about the "KIND=4", suggestiong use of ISO_FORTRAN_ENV. So maybe I should do that here. |
|
I've updated the branch to use int32. This requires updating the compiler flags to 2008 standard. There is some logic in the cmake file that tests the version; we are clearly always using at least 10.1.0 now so maybe this test can simply be removed? I have left it in for now though. |
Yes, I believe we can remove this test, as we should be on GNU 12 or above from now on. |
I'll open another PR for this, it probably shouldn't be done here. |
Fix #265.