[Beignet] [PATCH] gbe_bin_generator: fix the incorrect type of cl_internal_built_in_kernel_str_size.

Zhigang Gong zhigang.gong at intel.com
Wed Jun 25 08:23:53 PDT 2014


We should define it as size_t.

Signed-off-by: Zhigang Gong <zhigang.gong at intel.com>
---
 backend/src/gbe_bin_generater.cpp | 3 ++-
 src/cl_program.c                  | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/backend/src/gbe_bin_generater.cpp b/backend/src/gbe_bin_generater.cpp
index 2627673..925ba93 100644
--- a/backend/src/gbe_bin_generater.cpp
+++ b/backend/src/gbe_bin_generater.cpp
@@ -177,6 +177,7 @@ void program_build_instance::serialize_program(void) throw(int)
       if (last_slash != string::npos &&  last_dot != string::npos)
         array_name = bin_path.substr(last_slash + 1, last_dot - 1 - last_slash);
 
+      ofs << "#include <stddef.h>" << "\n";
       ofs << "char " << array_name << "[] = {" << "\n";
 
       sz = gbe_prog->serializeToBin(oss);
@@ -194,7 +195,7 @@ void program_build_instance::serialize_program(void) throw(int)
       ofs << "};\n";
 
       string array_size = array_name + "_size";
-      ofs << "int " << array_size << " = " << sz << ";" << "\n";
+      ofs << "size_t " << array_size << " = " << sz << ";" << "\n";
     } else {
       OUTF_UPDATE_SZ(header);
       OUTF_UPDATE_SZ(header);
diff --git a/src/cl_program.c b/src/cl_program.c
index 7888a8f..d7e9ae7 100644
--- a/src/cl_program.c
+++ b/src/cl_program.c
@@ -268,7 +268,7 @@ cl_program_create_with_built_in_kernles(cl_context     ctx,
   INVALID_DEVICE_IF (devices[0] != ctx->device);
 
   extern char cl_internal_built_in_kernel_str[];
-  extern int cl_internal_built_in_kernel_str_size;
+  extern size_t cl_internal_built_in_kernel_str_size;
   char* p_built_in_kernel_str =cl_internal_built_in_kernel_str;
   cl_int binary_status = CL_SUCCESS;
 
-- 
1.8.3.2



More information about the Beignet mailing list