Except the minor comment below, this patch is:

Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>

Sorry for this, the next time I'll build clover and learn from my mistakes. :-)

On 02/13/2016 06:01 PM, Serge Martin wrote:
---
  src/gallium/state_trackers/clover/core/kernel.cpp | 7 ++++---
  1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp 
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..a4ef2b1 100644
--- a/src/gallium/state_trackers/clover/core/kernel.cpp
+++ b/src/gallium/state_trackers/clover/core/kernel.cpp
@@ -76,9 +76,10 @@ kernel::launch(command_queue &q,
                                exec.g_buffers.data(), g_handles.data());

     // Fill information for the launch_grid() call.
-   info.block = pad_vector(q, block_size, 1).data(),
-   info.grid = pad_vector(q, reduced_grid_size, 1).data(),
-   info.pc = find(name_equals(_name), m.sysm).offset;
+   memcpy(info.block, pad_vector(q, block_size, 1).data(), sizeof(info.block));
+   memcpy(info.grid, pad_vector(q, reduced_grid_size, 1).data(),
+                                                            sizeof(info.grid));

Maybe you should fix alignment here.

+   info.pc = find(name_equals(_name), m.syms).offset;
     info.input = exec.input.data();

     q.pipe->launch_grid(q.pipe, &info);

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to