[Beignet] [PATCH] Fix based on piglit OpenCL falied case (cl-api-compile-program).

[Zhigang Gong]

This patch LGTM, will push latter, thanks.

On Tue, Dec 02, 2014 at 11:21:21AM +0800, Yan Wang wrote:
> 1. Return the expected error code.
> 2. Don't destroy cl_program object after comile error because it
> may be used still in the future.
> 
> Signed-off-by: Yan Wang <yan.wang at linux.intel.com>
> ---
>  src/cl_program.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/src/cl_program.c b/src/cl_program.c
> index fa67ef2..c30f85e 100644
> --- a/src/cl_program.c
> +++ b/src/cl_program.c
> @@ -738,9 +738,9 @@ cl_program_compile(cl_program            p,
>  
>      if (UNLIKELY(p->opaque == NULL)) {
>        if (p->build_log_sz > 0 && strstr(p->build_log, "error: error reading 'options'"))
> -        err = CL_INVALID_BUILD_OPTIONS;
> +        err = CL_INVALID_COMPILER_OPTIONS;
>        else
> -        err = CL_BUILD_PROGRAM_FAILURE;
> +        err = CL_COMPILE_PROGRAM_FAILURE;
>        goto error;
>      }
>  
> @@ -758,8 +758,6 @@ cl_program_compile(cl_program            p,
>  
>  error:
>    p->build_status = CL_BUILD_ERROR;
> -  cl_program_delete(p);
> -  p = NULL;
>    return err;
>  }
>  
> -- 
> 1.9.3
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet