On Fri, 2015-03-20 at 23:29 +0000, Tom Stellard wrote: > The storage size for local kernel args can be queried before the > arguments are set by using the CL_KERNEL_LOCAL_MEM_SIZE param > of clGetKernelWorkGroupInfo(). > > The spec says that if local kernel arguments have not been specified, > then we should assume their size is 0. > --- > src/gallium/state_trackers/clover/core/kernel.cpp | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp > b/src/gallium/state_trackers/clover/core/kernel.cpp > index 442762c..f4f8616 100644 > --- a/src/gallium/state_trackers/clover/core/kernel.cpp > +++ b/src/gallium/state_trackers/clover/core/kernel.cpp > @@ -399,6 +399,8 @@ kernel::global_argument::unbind(exec_context &ctx) { > > size_t > kernel::local_argument::storage() const { > + if (!_set) > + return 0;
Shouldn't _storage be 0 before anybody calls local_argument::set anyway? maybe initializing it in a constructor would suffice? jan > return _storage; > } > -- Jan Vesely <jan.ves...@rutgers.edu>
signature.asc
Description: This is a digitally signed message part
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev