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

storePhiOnParticles: fix misplaced pragma omp parallel if #5577

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

storePhiOnParticles: fix misplaced pragma omp parallel if #5577

wants to merge 1 commit into from
+3 −3

Conversation

lucafedeli88
Copy link
Member

This PR moves #pragma omp parallel if from an outer loop on levels down to the loop on tiles.
It seems to me that threads should be used for tiles rather than for levels here.

@lucafedeli88 lucafedeli88 added bug Something isn't working component: electrostatic electrostatic solver bug: affects latest release Bug also exists in latest release version labels Jan 20, 2025
@lucafedeli88 lucafedeli88 requested a review from RemiLehe January 20, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RemiLehe RemiLehe Awaiting requested review from RemiLehe

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
bug: affects latest release Bug also exists in latest release version bug Something isn't working component: electrostatic electrostatic solver
Projects
None yet
Milestone
No milestone
Development

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