From: yao <bitwangyaoyao@gmail.com>
Date: Tue, 4 Sep 2012 00:51:08 +0000 (+0800)
Subject: fix bug of ocl::canny
X-Git-Tag: accepted/2.0/20130307.220821~364^2~223^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8f8a69d6f5f7b9a17f313dcb9d9ef6f3fe56a0be;p=profile%2Fivi%2Fopencv.git

fix bug of ocl::canny
---

diff --git a/modules/ocl/src/kernels/imgproc_canny.cl b/modules/ocl/src/kernels/imgproc_canny.cl
index e32e928..65df508 100644
--- a/modules/ocl/src/kernels/imgproc_canny.cl
+++ b/modules/ocl/src/kernels/imgproc_canny.cl
@@ -100,15 +100,17 @@ __kernel
 
     __local int smem[16][18];
 
+    smem[lidy][lidx + 1] = src[gidx + gidy * src_step + src_offset]; 
+    if(lidx == 0)
+    {
+        smem[lidy][0]  = src[max(gidx - 1,  0)        + gidy * src_step + src_offset];
+        smem[lidy][17] = src[min(gidx + 16, cols - 1) + gidy * src_step + src_offset]; 
+    }
+    barrier(CLK_LOCAL_MEM_FENCE);
+
     if(gidy < rows)
     {
-        smem[lidy][lidx + 1] = src[gidx + gidy * src_step + src_offset]; 
-        if(lidx == 0)
-        {
-            smem[lidy][0]  = src[max(gidx - 1,  0)        + gidy * src_step + src_offset];
-            smem[lidy][17] = src[min(gidx + 16, cols - 1) + gidy * src_step + src_offset]; 
-        }
-        barrier(CLK_LOCAL_MEM_FENCE);
+
         if(gidx < cols)
         {
             dx_buf[gidx + gidy * dx_buf_step + dx_buf_offset] =
@@ -168,21 +170,21 @@ __kernel
 
     __local int sdx[18][16];
     __local int sdy[18][16];
-
-    if(gidx < cols)
+    
+    sdx[lidy + 1][lidx] = dx_buf[gidx + gidy * dx_buf_step + dx_buf_offset];
+    sdy[lidy + 1][lidx] = dy_buf[gidx + gidy * dy_buf_step + dy_buf_offset];
+    if(lidy == 0)
     {
-        sdx[lidy + 1][lidx] = dx_buf[gidx + gidy * dx_buf_step + dx_buf_offset];
-        sdy[lidy + 1][lidx] = dy_buf[gidx + gidy * dy_buf_step + dy_buf_offset];
-        if(lidy == 0)
-        {
-            sdx[0][lidx]  = dx_buf[gidx + max(gidy - 1,  0)        * dx_buf_step + dx_buf_offset];
-            sdx[17][lidx] = dx_buf[gidx + min(gidy + 16, rows - 1) * dx_buf_step + dx_buf_offset];
+        sdx[0][lidx]  = dx_buf[gidx + max(gidy - 1,  0)        * dx_buf_step + dx_buf_offset];
+        sdx[17][lidx] = dx_buf[gidx + min(gidy + 16, rows - 1) * dx_buf_step + dx_buf_offset];
 
-            sdy[0][lidx]  = dy_buf[gidx + max(gidy - 1,  0)        * dy_buf_step + dy_buf_offset];
-            sdy[17][lidx] = dy_buf[gidx + min(gidy + 16, rows - 1) * dy_buf_step + dy_buf_offset];
-        }
-        barrier(CLK_LOCAL_MEM_FENCE);
+        sdy[0][lidx]  = dy_buf[gidx + max(gidy - 1,  0)        * dy_buf_step + dy_buf_offset];
+        sdy[17][lidx] = dy_buf[gidx + min(gidy + 16, rows - 1) * dy_buf_step + dy_buf_offset];
+    }
+    barrier(CLK_LOCAL_MEM_FENCE);
 
+    if(gidx < cols)
+    {
         if(gidy < rows)
         {
             int x =  sdx[lidy][lidx] + 2 * sdx[lidy + 1][lidx] + sdx[lidy + 2][lidx];