Skip to content

Conversation

@pvelesko
Copy link
Collaborator

@pvelesko pvelesko commented Jul 19, 2024

Implement missing host math funcs

* Compiling the bitode library to host code leads to linking errors
some of the OpenCL natives being used in the library code are not
available for the CPU, resulting in linking errors

* OCML does not meant to be compiled in C++ mode
for this, we create a drop-in opencl header defining the missing
datatypes

* Since OCML uses headers in the same directory as the source files,
we must copy the necessary source files into a new directory so that
our replacement headers get picked up instead of the originals

* Implement a new header for mapping math funcs to host-side ocml

* Discover an additional way to trigger 128 bit error

* Discover a discrepancy in host and device func results

@pvelesko pvelesko force-pushed the missing-host-funcs branch 4 times, most recently from 49e81fd to b04d184 Compare July 23, 2024 10:58
@pvelesko pvelesko marked this pull request as ready for review July 23, 2024 11:37
pvelesko added 2 commits July 23, 2024 14:47
* Compiling the bitode library to host code leads to linking errors
some of the OpenCL natives being used in the library code are not
available for the CPU, resulting in linking errors

* OCML does not meant to be compiled in C++ mode
for this, we create a drop-in opencl header defining the missing
datatypes

* Since OCML uses headers in the same directory as the source files,
we must copy the necessary source files into a new directory so that
our replacement headers get picked up instead of the originals

* Implement a new header for mapping math funcs to host-side ocml

* Discover an additional way to trigger 128 bit error

* Discover a discrepancy in host and device func results
@pvelesko pvelesko force-pushed the missing-host-funcs branch from b04d184 to d3b4016 Compare July 23, 2024 11:47
@pvelesko pvelesko merged commit 29ce59f into main Jul 24, 2024
@pvelesko pvelesko deleted the missing-host-funcs branch July 24, 2024 05:32
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