-
Notifications
You must be signed in to change notification settings - Fork 3k
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
OMP pragma was added to support hetero_graph loop. #2148
OMP pragma was added to support hetero_graph loop. #2148
Conversation
2794db4
to
e3ac6b1
Compare
Could we only enable this optimization when the graph is stored on CPU? I don't think the optimization works for GPU as well. |
1105d3b
to
d624a99
Compare
@yzh119 Of course, done! |
d624a99
to
5a070b5
Compare
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.
I'm okay with this PR.
@yzh119 Thanks for your approval, could you please tell me whether it could be merged into the master branch? |
* OMP pragma was added to support hetero_graph loop, as follows: pragma omp parallel for * OMP pragma will be enabled iff the gpu is used for Co-authored-by: Zihao Ye <expye@outlook.com>
The OMP flag was added to support/obtain the inputs and outputs format of the matrices. This pull-request gives us the ability to get better performance in comparison with the master branch (approx. +5,6%). For brevity, the input and output format are triggered in a parallel way.