search

IANW-Projects / toolkitICL

toolkitICL: An open source tool for automated OpenCL kernel execution and profiling.
8 stars 3 forks source link

Add OpenCL platform? #6

Closed philipheinisch closed 5 years ago

philipheinisch commented 5 years ago

Based on the suggestion by @ranocha: Does it make sense so add the OpenCL platform name to the HDF5 output?

Does the platform name provide any more information compared to device name and OpenCL version? Or should we maybe implement a new structure, that joins all the necessary information in a single HDF5 struct or string?

ranocha commented 5 years ago

I think it would be a bit more clear which platform is used, e.g. POCL, Intel Runtime, Nvidia etc. It can also be derived from the deice name and OpenCL version, but some additional redundancy is often good to avoid confusion. For me, these three pieces of information would be enough and an additional struct would not be necessary.

philipheinisch commented 5 years ago

Ok, than I will add the feature.