Skip to content
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

First run of OpenCL is ready to go #8

Merged
merged 6 commits into from
Sep 23, 2011
Merged

First run of OpenCL is ready to go #8

merged 6 commits into from
Sep 23, 2011

Conversation

mdsmith
Copy link
Contributor

@mdsmith mdsmith commented Sep 22, 2011

Functional OpenCL ML evaluation with appropriate cmake build options and a separate .cl file for kernels. Lance helped with all of the cmake and .cl file separation, so he knows what is going on (and its state). Seems to be working fine, I'll get on making it pretty, commented, and faster.

Biggest TODO: the old build scripts not only included all necessary
frameworks/libraries, they had the platform detection code and
defined certain variables for later use. This needs to be integrated
into the new build scheme.

Things probably wont stay as they are, however. It would be nice
to generalize out the functions in this calculation, or at least
isolate them to their own library. We'll see.

- in calcnode.h: the header information for the likefuncocl objects
- in likefunc.cpp: the setup, teardown and kernel launching function
calls
- in likefuncocl.cpp The kernels and their related functions.
I need to get the cmake support file from the veg/hyphy, so I'm
going to do some git magic to get that down here.
nlhepler added a commit that referenced this pull request Sep 23, 2011
First run of OpenCL is ready to go
@nlhepler nlhepler merged commit c88e54d into veg:master Sep 23, 2011
mdsmith pushed a commit to mdsmith/hyphy that referenced this pull request Jul 3, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants