Utest: Change the kernel index to fit case index

Submitted by Pan Xiuli on Sept. 12, 2016, 7:27 a.m.

Details

Message ID 1473665274-28262-1-git-send-email-xiuli.pan@intel.com
State New
Headers show
Series "Utest: Change the kernel index to fit case index" ( rev: 1 ) in Beignet

Not browsing as part of any series.

Commit Message

Pan Xiuli Sept. 12, 2016, 7:27 a.m.
From: Pan Xiuli <xiuli.pan@intel.com>

Sub group broadcast case will fail in simd 16, for the kernel doesn't
fit the test case.

Signed-off-by: Pan Xiuli <xiuli.pan@intel.com>
---
 kernels/compiler_subgroup_broadcast.cl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/kernels/compiler_subgroup_broadcast.cl b/kernels/compiler_subgroup_broadcast.cl
index 457c826..63e9568 100644
--- a/kernels/compiler_subgroup_broadcast.cl
+++ b/kernels/compiler_subgroup_broadcast.cl
@@ -9,7 +9,7 @@  kernel void compiler_subgroup_broadcast_imm_int(global int *src,
   uint index = get_global_id(0);
 
   int val = src[index];
-  int broadcast_val = sub_group_broadcast(val, 10);
+  int broadcast_val = sub_group_broadcast(val, 2);
   dst[index] = broadcast_val;
 }
 kernel void compiler_subgroup_broadcast_int(global int *src,

Comments

LGTM, thanks, pushed.

> -----Original Message-----

> From: Beignet [mailto:beignet-bounces@lists.freedesktop.org] On Behalf Of

> Xiuli Pan

> Sent: Monday, September 12, 2016 15:28

> To: beignet@lists.freedesktop.org

> Cc: Pan, Xiuli <xiuli.pan@intel.com>

> Subject: [Beignet] [PATCH] Utest: Change the kernel index to fit case index

> 

> From: Pan Xiuli <xiuli.pan@intel.com>

> 

> Sub group broadcast case will fail in simd 16, for the kernel doesn't fit the test

> case.

> 

> Signed-off-by: Pan Xiuli <xiuli.pan@intel.com>

> ---

>  kernels/compiler_subgroup_broadcast.cl | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/kernels/compiler_subgroup_broadcast.cl

> b/kernels/compiler_subgroup_broadcast.cl

> index 457c826..63e9568 100644

> --- a/kernels/compiler_subgroup_broadcast.cl

> +++ b/kernels/compiler_subgroup_broadcast.cl

> @@ -9,7 +9,7 @@ kernel void

> compiler_subgroup_broadcast_imm_int(global int *src,

>    uint index = get_global_id(0);

> 

>    int val = src[index];

> -  int broadcast_val = sub_group_broadcast(val, 10);

> +  int broadcast_val = sub_group_broadcast(val, 2);

>    dst[index] = broadcast_val;

>  }

>  kernel void compiler_subgroup_broadcast_int(global int *src,

> --

> 2.7.4

> 

> _______________________________________________

> Beignet mailing list

> Beignet@lists.freedesktop.org

> https://lists.freedesktop.org/mailman/listinfo/beignet
LGTM, pushed, thanks.

> -----Original Message-----

> From: Beignet [mailto:beignet-bounces@lists.freedesktop.org] On Behalf Of

> Xiuli Pan

> Sent: Monday, September 12, 2016 15:28

> To: beignet@lists.freedesktop.org

> Cc: Pan, Xiuli <xiuli.pan@intel.com>

> Subject: [Beignet] [PATCH] Utest: Change the kernel index to fit case index

> 

> From: Pan Xiuli <xiuli.pan@intel.com>

> 

> Sub group broadcast case will fail in simd 16, for the kernel doesn't fit the test

> case.

> 

> Signed-off-by: Pan Xiuli <xiuli.pan@intel.com>

> ---

>  kernels/compiler_subgroup_broadcast.cl | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/kernels/compiler_subgroup_broadcast.cl

> b/kernels/compiler_subgroup_broadcast.cl

> index 457c826..63e9568 100644

> --- a/kernels/compiler_subgroup_broadcast.cl

> +++ b/kernels/compiler_subgroup_broadcast.cl

> @@ -9,7 +9,7 @@ kernel void

> compiler_subgroup_broadcast_imm_int(global int *src,

>    uint index = get_global_id(0);

> 

>    int val = src[index];

> -  int broadcast_val = sub_group_broadcast(val, 10);

> +  int broadcast_val = sub_group_broadcast(val, 2);

>    dst[index] = broadcast_val;

>  }

>  kernel void compiler_subgroup_broadcast_int(global int *src,

> --

> 2.7.4

> 

> _______________________________________________

> Beignet mailing list

> Beignet@lists.freedesktop.org

> https://lists.freedesktop.org/mailman/listinfo/beignet