[Beignet] [PATCH] Fix several typos in unit test.

Zhigang Gong zhigang.gong at linux.intel.com
Fri Jun 21 01:11:28 PDT 2013


Pushed, thanks.

On Wed, Jun 19, 2013 at 10:04:54AM +0800, Ruiling Song wrote:
> compiler_sub_bytes and compiler_sub_shorts
> 
> Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> ---
>  utests/compiler_sub_bytes.cpp  |    4 ++--
>  utests/compiler_sub_shorts.cpp |    4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/utests/compiler_sub_bytes.cpp b/utests/compiler_sub_bytes.cpp
> index 49a5261..740a8fd 100644
> --- a/utests/compiler_sub_bytes.cpp
> +++ b/utests/compiler_sub_bytes.cpp
> @@ -11,7 +11,7 @@ static void compiler_sub_bytes(void)
>    for (uint32_t i = 0; i < n; ++i) ((int8_t*)buf_data[0])[i] = (int8_t) rand();
>    for (uint32_t i = 0; i < n; ++i) ((int8_t*)buf_data[1])[i] = (int8_t) rand();
>    OCL_CREATE_BUFFER(buf[0], CL_MEM_COPY_HOST_PTR, n * sizeof(int8_t), buf_data[0]);
> -  OCL_CREATE_BUFFER(buf[1], CL_MEM_COPY_HOST_PTR, n * sizeof(int8_t), buf_data[0]);
> +  OCL_CREATE_BUFFER(buf[1], CL_MEM_COPY_HOST_PTR, n * sizeof(int8_t), buf_data[1]);
>    OCL_CREATE_BUFFER(buf[2], 0, n * sizeof(int8_t), NULL);
>  
>    // Run the kernel
> @@ -25,7 +25,7 @@ static void compiler_sub_bytes(void)
>    // Check result
>    OCL_MAP_BUFFER(2);
>    for (uint32_t i = 0; i < n; ++i)
> -    OCL_ASSERT(((int8_t*)buf_data[2])[i] = ((int8_t*)buf_data[0])[i] - ((int8_t*)buf_data[1])[i]);
> +    OCL_ASSERT(((int8_t*)buf_data[2])[i] == (int8_t)(((int8_t*)buf_data[0])[i] - ((int8_t*)buf_data[1])[i]));
>    free(buf_data[0]);
>    free(buf_data[1]);
>    buf_data[0] = buf_data[1] = NULL;
> diff --git a/utests/compiler_sub_shorts.cpp b/utests/compiler_sub_shorts.cpp
> index 4aeeca3..7c24a56 100644
> --- a/utests/compiler_sub_shorts.cpp
> +++ b/utests/compiler_sub_shorts.cpp
> @@ -11,7 +11,7 @@ static void compiler_sub_shorts(void)
>    for (uint32_t i = 0; i < n; ++i) ((int16_t*)buf_data[0])[i] = (int16_t) rand();
>    for (uint32_t i = 0; i < n; ++i) ((int16_t*)buf_data[1])[i] = (int16_t) rand();
>    OCL_CREATE_BUFFER(buf[0], CL_MEM_COPY_HOST_PTR, n * sizeof(int16_t), buf_data[0]);
> -  OCL_CREATE_BUFFER(buf[1], CL_MEM_COPY_HOST_PTR, n * sizeof(int16_t), buf_data[0]);
> +  OCL_CREATE_BUFFER(buf[1], CL_MEM_COPY_HOST_PTR, n * sizeof(int16_t), buf_data[1]);
>    OCL_CREATE_BUFFER(buf[2], 0, n * sizeof(int16_t), NULL);
>  
>    // Run the kernel
> @@ -25,7 +25,7 @@ static void compiler_sub_shorts(void)
>    // Check result
>    OCL_MAP_BUFFER(2);
>    for (uint32_t i = 0; i < n; ++i)
> -    OCL_ASSERT(((int16_t*)buf_data[2])[i] = ((int16_t*)buf_data[0])[i] - ((int16_t*)buf_data[1])[i]);
> +    OCL_ASSERT(((int16_t*)buf_data[2])[i] == (int16_t)(((int16_t*)buf_data[0])[i] - ((int16_t*)buf_data[1])[i]));
>    free(buf_data[0]);
>    free(buf_data[1]);
>    buf_data[0] = buf_data[1] = NULL;
> -- 
> 1.7.9.5
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list