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

Add ftarget #63

Merged
merged 1 commit into from
Sep 15, 2023
Merged

Add ftarget #63

merged 1 commit into from
Sep 15, 2023

Conversation

jschueller
Copy link
Collaborator

No description provided.

@zaikunzhang
Copy link
Member

Hi @jschueller Thank you so much for your very impressively prompt action!

@zaikunzhang
Copy link
Member

Hi @jschueller ,

What is the meaning of the value attribute? When to use it and when not? I did not know it before. See the following for example.

real(C_DOUBLE), intent(in) :: xl(n)                                                                                                  
real(C_DOUBLE), intent(in), value :: ftarget                                                                                          

Thanks.

Zaikun

@jschueller
Copy link
Collaborator Author

the argument is passed by value instead of by reference (pointer)

@zaikunzhang
Copy link
Member

the argument is passed by value instead of by reference (pointer)

It seems that we can only apply it to “scalars”, not arrays?

@jschueller
Copy link
Collaborator Author

yes

@zaikunzhang
Copy link
Member

Great. Everything looks fine. I made a few small changes regarding the workflow files. Would you mind synchronizing them before we merge the PR? Thank you very much.

@jschueller
Copy link
Collaborator Author

done

@zaikunzhang zaikunzhang merged commit b1f467b into libprima:main Sep 15, 2023
@jschueller jschueller deleted the ftarget branch September 15, 2023 11:37
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.

2 participants