We could change the signature of the function.  However, it would
require changing gdb_target_read in jit-reader.h, which is an exported
interface.  It's probably better to just add a cast in our code than to
break other people's code.
gdb/ChangeLog:
	* jit.c (jit_target_read_impl): Add cast.
+2015-10-29  Simon Marchi  <simon.marchi@polymtl.ca>
+
+       * jit.c (jit_target_read_impl): Add cast.
+
 2015-10-29  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * jit.c (jit_reader_load): Add cast.
 
 static enum gdb_status
 jit_target_read_impl (GDB_CORE_ADDR target_mem, void *gdb_buf, int len)
 {
-  int result = target_read_memory ((CORE_ADDR) target_mem, gdb_buf, len);
+  int result = target_read_memory ((CORE_ADDR) target_mem,
+                                  (gdb_byte *) gdb_buf, len);
   if (result == 0)
     return GDB_SUCCESS;
   else