Tom Stellard <[email protected]> writes: > v2: > - Report correct values for CL_DEVICE_NATIVE_VECTOR_WIDTH_DOUBLE > and CL_DEVICE_PREFERRED_VECTOR_WIDTH_DOUBLE. > - Only define cl_khr_fp64 if the extension is supported. > - Remove trailing space from extension string. > - Rename device query function from cl_khr_fp86() to > has_doubles(). > > v3: > - Return 0 for device::doubled_fp_confg() when doubles aren't > supported. > > v4: > - Remove device query for double fp_config.
Reviewed-by: Francisco Jerez <[email protected]> > --- > src/gallium/state_trackers/clover/api/device.cpp | 21 ++++++++++++++++++--- > src/gallium/state_trackers/clover/core/device.cpp | 6 ++++++ > src/gallium/state_trackers/clover/core/device.hpp | 1 + > 3 files changed, 25 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/state_trackers/clover/api/device.cpp > b/src/gallium/state_trackers/clover/api/device.cpp > index e825468..b1f556f 100644 > --- a/src/gallium/state_trackers/clover/api/device.cpp > +++ b/src/gallium/state_trackers/clover/api/device.cpp > @@ -145,7 +145,7 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, > break; > > case CL_DEVICE_PREFERRED_VECTOR_WIDTH_DOUBLE: > - buf.as_scalar<cl_uint>() = 2; > + buf.as_scalar<cl_uint>() = dev.has_doubles() ? 2 : 0; > break; > > case CL_DEVICE_PREFERRED_VECTOR_WIDTH_HALF: > @@ -205,6 +205,21 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, > CL_FP_DENORM | CL_FP_INF_NAN | CL_FP_ROUND_TO_NEAREST; > break; > > + case CL_DEVICE_DOUBLE_FP_CONFIG: > + if (dev.has_doubles()) > + // This is the "mandated minimum double precision floating-point > + // capability" > + buf.as_scalar<cl_device_fp_config>() = > + CL_FP_FMA > + | CL_FP_ROUND_TO_NEAREST > + | CL_FP_ROUND_TO_ZERO > + | CL_FP_ROUND_TO_INF > + | CL_FP_INF_NAN > + | CL_FP_DENORM; > + else > + buf.as_scalar<cl_device_fp_config>() = 0; > + break; > + > case CL_DEVICE_GLOBAL_MEM_CACHE_TYPE: > buf.as_scalar<cl_device_mem_cache_type>() = CL_NONE; > break; > @@ -283,7 +298,7 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, > break; > > case CL_DEVICE_EXTENSIONS: > - buf.as_string() = ""; > + buf.as_string() = dev.has_doubles() ? "cl_khr_fp64" : ""; > break; > > case CL_DEVICE_PLATFORM: > @@ -315,7 +330,7 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, > break; > > case CL_DEVICE_NATIVE_VECTOR_WIDTH_DOUBLE: > - buf.as_scalar<cl_uint>() = 2; > + buf.as_scalar<cl_uint>() = dev.has_doubles() ? 2 : 0; > break; > > case CL_DEVICE_NATIVE_VECTOR_WIDTH_HALF: > diff --git a/src/gallium/state_trackers/clover/core/device.cpp > b/src/gallium/state_trackers/clover/core/device.cpp > index 688a7dd..c3f3b4e 100644 > --- a/src/gallium/state_trackers/clover/core/device.cpp > +++ b/src/gallium/state_trackers/clover/core/device.cpp > @@ -173,6 +173,12 @@ device::image_support() const { > PIPE_COMPUTE_CAP_IMAGES_SUPPORTED)[0]; > } > > +bool > +device::has_doubles() const { > + return pipe->get_shader_param(pipe, PIPE_SHADER_COMPUTE, > + PIPE_SHADER_CAP_DOUBLES); > +} > + > std::vector<size_t> > device::max_block_size() const { > auto v = get_compute_param<uint64_t>(pipe, > PIPE_COMPUTE_CAP_MAX_BLOCK_SIZE); > diff --git a/src/gallium/state_trackers/clover/core/device.hpp > b/src/gallium/state_trackers/clover/core/device.hpp > index 2201700..de5fc6b 100644 > --- a/src/gallium/state_trackers/clover/core/device.hpp > +++ b/src/gallium/state_trackers/clover/core/device.hpp > @@ -64,6 +64,7 @@ namespace clover { > cl_uint max_clock_frequency() const; > cl_uint max_compute_units() const; > bool image_support() const; > + bool has_doubles() const; > > std::vector<size_t> max_block_size() const; > std::string device_name() const; > -- > 2.0.4
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
