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 overload of makeDistArray that takes an array #2757

Closed
bmcdonald3 opened this issue Sep 8, 2023 · 0 comments · Fixed by #2758
Closed

Add overload of makeDistArray that takes an array #2757

bmcdonald3 opened this issue Sep 8, 2023 · 0 comments · Fixed by #2758
Assignees

Comments

@bmcdonald3
Copy link
Contributor

Previously when creating a copy of an array using createSymEntry we were always creating the array first and then assigning, which meant that the arrays were going to be default initializing, resulting in a performance hit.

Adding an overload of makeDistArray that takes an array and does not default initialize in that case resolves this performance hit prior to Chapel 1.31, but since there is no way to avoid default initialization, this is not fixed in 1.32 just yet, but we are avoiding an extra domain creation, which will improve performance slightly in 1.32.

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 a pull request may close this issue.

1 participant