-
Notifications
You must be signed in to change notification settings - Fork 52
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
Fix gfx8 opencl #97
base: amd-staging
Are you sure you want to change the base?
Fix gfx8 opencl #97
Conversation
This condition was added when we supported PAL openCL on gfx8, but when ROC_ENABLE_PRE_VEGA was dropped and PAL OpenCL on Linux was deprecated, this logic should have been dropped completely. Change-Id: I0a68ef4723d9ebde7d958c58dbb0b6991f69d817 Signed-off-by: Jeremy Newton <[email protected]>
Good day, I am a
I would like to help you with the tests, is there anything I can do? |
@Germano0 gfx8 isn't really supported by the ROCm team anymore, but there's no harm in enabling it in my opinion. |
Key Points:
Recap of Changes Made:
Tested RX580 6.11.6-arch1-1 Platform Name AMD Accelerated Parallel Processing NULL platform behavior ICD loader properties https://chatgpt.com/share/674405a9-d344-8010-828b-70054d2429ee |
This condition was added when we supported PAL openCL on gfx8, but when ROC_ENABLE_PRE_VEGA was dropped and PAL OpenCL on Linux was deprecated, this logic should have been dropped completely.