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.w...@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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet

Reply via email to