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.

Sign up for GitHub

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

Jump to bottom

Loop adapted to strict compiler scrutiny #404

Closed
wants to merge 7 commits into from
Closed

Loop adapted to strict compiler scrutiny #404

wants to merge 7 commits into from

Conversation

quickfly
Copy link
Contributor

@quickfly quickfly commented Sep 6, 2024

in DIMATCOPY opt out case. (Was previously traditional fortran practice.)

@quickfly
Loop adapted to strict compiler scrutiny
d0a5243 
in DIMATCOPY opt out case. (Was previously traditional
fortran practice.)
@quickfly
Copy link
Contributor Author

quickfly commented Sep 6, 2024

Hi Igor,
thanks for the Info!
I have corrected the issue (hopefully) and pushed it to the landau branch.
(Explanation for the loops: The transposition loops were a little simpler to formulate in the traditional fortran style without strict index bounds. MKL and BLAS routines, if chosen by setting the Macro DIMATCOPY in the platform files, don't care about these bounds anyway.)
Could you please merge it back into master, if you're ok with the changes?
Best,
Klaus

@quickfly quickfly closed this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@quickfly