[Beignet] [PATCH] utest: correct a typo in compiler_cl_finish.cpp

Yang, Rong R rong.r.yang at intel.com
Sun Jan 24 19:25:56 PST 2016


LGTM, pushed, thanks.

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Guo Yejun
> Sent: Monday, January 25, 2016 3:32
> To: beignet at lists.freedesktop.org
> Cc: Guo, Yejun
> Subject: [Beignet] [PATCH] utest: correct a typo in compiler_cl_finish.cpp
> 
> there is no logical relationship between the time of finish and map, remove
> the condition.
> 
> Signed-off-by: Guo Yejun <yejun.guo at intel.com>
> ---
>  utests/compiler_cl_finish.cpp | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/utests/compiler_cl_finish.cpp b/utests/compiler_cl_finish.cpp
> index 7c7dee3..c637ecc 100644
> --- a/utests/compiler_cl_finish.cpp
> +++ b/utests/compiler_cl_finish.cpp
> @@ -9,7 +9,7 @@ static void compiler_cl_finish(void)  {
>    const size_t n = 16*1024*1024;
>    struct timeval t1, t2;
> -  float t_fin, t_map_w_fin,t_map_wo_fin;
> +  float t_map_w_fin,t_map_wo_fin;
> 
>    // Setup kernel and buffers
>    OCL_CREATE_KERNEL("test_cl_finish");
> @@ -26,10 +26,7 @@ static void compiler_cl_finish(void)
> 
>    // 1st time map after clFinish
>    OCL_NDRANGE(1);
> -  T_GET(t1);
>    OCL_FINISH();
> -  T_GET(t2);
> -  t_fin = T_LAPSE(t1, t2);
> 
>    T_GET(t1);
>    OCL_MAP_BUFFER(0);
> @@ -43,7 +40,7 @@ static void compiler_cl_finish(void)
>    T_GET(t2);
>    t_map_wo_fin = T_LAPSE(t1, t2);
> 
> -  OCL_ASSERT(t_fin > t_map_w_fin && t_map_wo_fin > t_map_w_fin);
> +  OCL_ASSERT(t_map_wo_fin > t_map_w_fin);
>    OCL_UNMAP_BUFFER(0);
>  }
> 
> --
> 1.9.1
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list