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 parameter to force using GlobalMemory launch mechanism using HIP #5796

Merged
merged 7 commits into from
Jan 24, 2023

Conversation

Rombur
Copy link
Member

@Rombur Rombur commented Jan 23, 2023

Add a parameter that users can use to force a kernel to be launched using GlobalMemory. This is sometimes necessary due to compiler bugs in ROCm 5.3 and ROCm 5.4 see #5606 and #5690

cc: @arghdos

core/unit_test/TestViewAPI.hpp Outdated Show resolved Hide resolved
core/unit_test/TestViewAPI_d.hpp Outdated Show resolved Hide resolved
Co-authored-by: Damien L-G <dalg24+github@gmail.com>
Copy link
Member

@crtrott crtrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we name it ImplForceGlobalLaunch make it clearer that it has something to do with launch mechanism?

@PhilMiller PhilMiller changed the title Add parameter to force using GlobaLMemory launch mechanism using HIP Add parameter to force using GlobalMemory launch mechanism using HIP Jan 23, 2023
@Rombur
Copy link
Member Author

Rombur commented Jan 23, 2023

done

@crtrott crtrott merged commit a75c613 into kokkos:develop Jan 24, 2023
@dalg24
Copy link
Member

dalg24 commented Jan 24, 2023

@Rombur plz cherry-pick into RC 4.0

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.

None yet

3 participants