Ben Ashbaugh

Removing "focused review" for now - not quite ready yet.

@gwawiork do you have all of the other fixes from #1642? I'm trying to figure out if these are known failures or new failures. Thanks!

One item we briefly discussed in last week's teleconference: We recently added the ability to have "loader extensions" (see the ICD loader info PR https://github.com/KhronosGroup/OpenCL-ICD-Loader/pull/188). Could we add a similar...

Hi, the `OPENCL_ICD_LOADER_HEADERS_DIR` should be where you've cloned the OpenCL headers, and does not need the `CL` suffix. Also, this should be a CMake variable, not a shell environment variable....

Hi @ggghamd, can you please confirm that you were able to build the ICD loader following the instructions above? Thanks!

This seems like a reasonable thing to do. I'd personally prefer to put the links to C++ packages towards the bottom of the README vs. the top, but I could...

> * Do you mind if I pick-up where you left it No, not at all! I'm also happy to help if needed, just let me know what I can...

I wrote up some notes how to run the ICD loader tests on Linux a while back. Here's what I do: > To run the tests, execute the tester using...

First off - I think this is an interesting capability and we have seen requests for something like it, so thank you @manycoresoft for doing the work to get this...

Darn, I thought we fixed these with https://github.com/KhronosGroup/OpenCL-Docs/pull/983, but it looks like we only did so for the CL_VERSION_1_X enums and not for the newer 2.x or 3.0 enums. Good...