-
Notifications
You must be signed in to change notification settings - Fork 113
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
Extension of cl_khr_icd to 2.0.0 with loader managed dispatch. #1005
base: main
Are you sure you want to change the base?
Conversation
44dc10e
to
cc8ad1b
Compare
@keryell was concerned clPlatformID pointer for a non ICD 2.0.0 compatible could have the same value as the tag. One way to avoid this problem would be to have a second entry in the dispatch table set to the tag value (clUnloadCompiler comes to mind) and check that both of these have the tag while loading the library. If the tag were not present in both, the implementation would not be loaded. |
Thanks. This is quite simpler and more portable than the solution I was thinking to. |
cc8ad1b
to
4ec65bf
Compare
This is a formal description of the proposal discussed here:
#1003