Pekka Jääskeläinen
Pekka Jääskeläinen
Right. Should be fixed in PoCL side via include_next or similar?
The problem is that the build uses the installed headers by default. Check HAVE_OPENCL_H variable in the main CMakeLists.txt. This drops the $SRCDIR/include from the search paths which causes it...
> Why are we looking at system OpenCL headers in the first place? If building requires bleeding edge headers anyway that seems mainly like a source for trouble. So unless...
@franz do you remember the motivations for this build time option of using or not using the system headers (for the build)?
We discussed this here with Michal, he pointed that this is due to ICD loader. If we have < 3.0 ICD loader installed we cannot use 3.0 APIs and we...
573 - conformance_basic_micro_memory (Failed) fails here (LLVM 14), at least in my first trial (these could be race conditions). Log: ``` 573/651 Testing: conformance_basic_micro_memory 573/651 Test: conformance_basic_micro_memory Command: "/home/visit0r/src/pocl/test-pr-cmake-llvm-14-build/examples/conformance/src/conformance-build/test_conformance/basic/test_basic" "vload_global"...
https://www.khronos.org/registry/OpenCL/specs/opencl-1.2.pdf 6.5.2 specifically disallows this case. Lifting that standard restriction might require kernel compiler updates, unless already implemented in upstream LLVM. I'm surprised other SDKs allow that without errors (do...
I'm not sure I understand the details here. Is this problem only when using a non-ICD build since doesn't the ICD loader open libpocl.so with dlopen() directly via the .icd...
What's the plan with this work?
Looked good and worked with my local test envs, so I pushed. Thanks! Although I wonder are there test cases that hit this code path in the repo now, are...