Does not build on OS X due to missing OpenCL C++ bindings.

krrishnarraj / clpeak

A tool which profiles OpenCL devices to find their peak capacities

Apache License 2.0

396 stars 111 forks source link

Closed he-sk closed 8 years ago

he-sk commented 8 years ago

OS X El Capitan does not ship the cl.hpp headers. I think the license permits to just include this file in the sources.

krrishnarraj commented 8 years ago

I have made some changes. Could you check now with devel branch and confirm

he-sk commented 8 years ago

I've created a pull request based on the devel branch.

he-sk commented 8 years ago

Just tested this on OS X and while it does compile, it hangs after printing the version info.

The master branch worked yesterday. So, some commit in the devel branch breaks the program on OS X.

I'll investigate later.

he-sk commented 8 years ago

Seems like commit 2aa9805e (remove platform specific disables) is the culprit. If I back it out it works on OS X. Any idea what might cause the hang?

krrishnarraj commented 8 years ago

integer kernels were previously disabled due to compilation problem in apple platform. Thought would be fixed by now. sigh! I will revert that commit

krrishnarraj commented 8 years ago

Could you check now if devel branch is clean on apple?

Thanks BTW

he-sk commented 8 years ago

It works now. I've created a pull request with a report on Apple Iris.