search for: grid_offset

...e9..dad66aa 100644 --- a/src/gallium/state_trackers/clover/core/kernel.cpp +++ b/src/gallium/state_trackers/clover/core/kernel.cpp @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, const auto reduced_grid_size = map(divides(), grid_size, block_size); void *st = exec.bind(&q, grid_offset); - struct pipe_grid_info info; + struct pipe_grid_info info = { 0, }; // The handles are created during exec_context::bind(), so we need make // sure to call exec_context::bind() before retrieving them. -- 2.7.2

...ers/clover/core/kernel.cpp >> +++ b/src/gallium/state_trackers/clover/core/kernel.cpp >> @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, >> const auto reduced_grid_size = >> map(divides(), grid_size, block_size); >> void *st = exec.bind(&q, grid_offset); >> - struct pipe_grid_info info; >> + struct pipe_grid_info info = { 0, }; > > Right, good catch, it's my fault. > > = { 0 }; is enough btw. I prefer to add the "," to make clear that we are initializing the entire struct, I read it as ", ..."...

...e9..1ab87ec 100644 --- a/src/gallium/state_trackers/clover/core/kernel.cpp +++ b/src/gallium/state_trackers/clover/core/kernel.cpp @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, const auto reduced_grid_size = map(divides(), grid_size, block_size); void *st = exec.bind(&q, grid_offset); - struct pipe_grid_info info; + struct pipe_grid_info info = { 0 }; // The handles are created during exec_context::bind(), so we need make // sure to call exec_context::bind() before retrieving them. -- 2.7.2

...e9..1ab87ec 100644 --- a/src/gallium/state_trackers/clover/core/kernel.cpp +++ b/src/gallium/state_trackers/clover/core/kernel.cpp @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, const auto reduced_grid_size = map(divides(), grid_size, block_size); void *st = exec.bind(&q, grid_offset); - struct pipe_grid_info info; + struct pipe_grid_info info = { 0 }; // The handles are created during exec_context::bind(), so we need make // sure to call exec_context::bind() before retrieving them. -- 2.7.2

...gt; +++ b/src/gallium/state_trackers/clover/core/kernel.cpp >>>> @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, >>>> const auto reduced_grid_size = >>>> map(divides(), grid_size, block_size); >>>> void *st = exec.bind(&q, grid_offset); >>>> - struct pipe_grid_info info; >>>> + struct pipe_grid_info info = { 0, }; >>> >>> Right, good catch, it's my fault. >>> >>> = { 0 }; is enough btw. >> >> I prefer to add the "," to make clear that we ar...

.../gallium/state_trackers/clover/core/kernel.cpp > +++ b/src/gallium/state_trackers/clover/core/kernel.cpp > @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, > const auto reduced_grid_size = > map(divides(), grid_size, block_size); > void *st = exec.bind(&q, grid_offset); > - struct pipe_grid_info info; > + struct pipe_grid_info info = { 0, }; Right, good catch, it's my fault. = { 0 }; is enough btw. This should be backported to mesa 11.2 I guess, could you please send a v2 with this minor fix and add the cc thing? Thanks. > > // The...

...el.cpp >>> +++ b/src/gallium/state_trackers/clover/core/kernel.cpp >>> @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, >>> const auto reduced_grid_size = >>> map(divides(), grid_size, block_size); >>> void *st = exec.bind(&q, grid_offset); >>> - struct pipe_grid_info info; >>> + struct pipe_grid_info info = { 0, }; >> >> Right, good catch, it's my fault. >> >> = { 0 }; is enough btw. > > I prefer to add the "," to make clear that we are initializing the > entire s...

...llium/state_trackers/clover/core/kernel.cpp >>>>> @@ -55,7 +55,7 @@ kernel::launch(command_queue &q, >>>>> const auto reduced_grid_size = >>>>> map(divides(), grid_size, block_size); >>>>> void *st = exec.bind(&q, grid_offset); >>>>> - struct pipe_grid_info info; >>>>> + struct pipe_grid_info info = { 0, }; >>>> >>>> Right, good catch, it's my fault. >>>> >>>> = { 0 }; is enough btw. >>> >>> I prefer to add the "...