-
Notifications
You must be signed in to change notification settings - Fork 127
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 continuous integration #62
Comments
Merged
kpet
added a commit
that referenced
this issue
Sep 16, 2019
Since we build with CL_HPP_MINIMUM_OPENCL_VERSION defined to 100, the bindings are querying the platform to decide whether to use clCreateCommandQueue or clCreateCommandQueueWithProperties. We thus need to make the stub function for clGetPlatformInfo report 2.0 when building with CL_HPP_TARGET_OPENCL_VERSION >= 200 to mirror the queue creation function check. Contributes to #62. Signed-off-by: Kevin Petit <kevin.petit@arm.com>
kpet
added a commit
that referenced
this issue
Sep 16, 2019
Since we build with CL_HPP_MINIMUM_OPENCL_VERSION defined to 100, the bindings are querying the platform to decide whether to use clCreateCommandQueue or clCreateCommandQueueWithProperties. We thus need to make the stub function for clGetPlatformInfo report 2.0 when building with CL_HPP_TARGET_OPENCL_VERSION >= 200 to mirror the queue creation function check. Also allow calls to clCreateContextFromType with no properties on Apple platforms to mirror the logic in the bindings. Contributes to #62. Signed-off-by: Kevin Petit <kevin.petit@arm.com>
bashbaug
pushed a commit
that referenced
this issue
Sep 20, 2019
Since we build with CL_HPP_MINIMUM_OPENCL_VERSION defined to 100, the bindings are querying the platform to decide whether to use clCreateCommandQueue or clCreateCommandQueueWithProperties. We thus need to make the stub function for clGetPlatformInfo report 2.0 when building with CL_HPP_TARGET_OPENCL_VERSION >= 200 to mirror the queue creation function check. Also allow calls to clCreateContextFromType with no properties on Apple platforms to mirror the logic in the bindings. Contributes to #62. Signed-off-by: Kevin Petit <kevin.petit@arm.com>
We now have basic continuous integration up and running. I'm going to close this. I've created #78 to try and capture a detailed list of the configurations that we want to test. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: