Fix some compiler warnings with -pedantic

diff --git a/src/mesa/pipe/tgsi/exec/tgsi_sse2.c b/src/mesa/pipe/tgsi/exec/tgsi_sse2.c
index e7a6112e6e8799b52c2af08f580b5152337e493f..d840d64cae49958ece773f57050f784bf33ae8e8 100755 (executable)
--- a/src/mesa/pipe/tgsi/exec/tgsi_sse2.c
+++ b/src/mesa/pipe/tgsi/exec/tgsi_sse2.c
@@ -408,7 +408,7 @@ emit_func_call1(
    struct x86_function *func,
    unsigned xmm_dst,
    unsigned storage,
-   unsigned char *code )
+   void (*code)() )
 {
    x86_push(
       func,
@@ -443,7 +443,7 @@ emit_func_call2(
    unsigned xmm_dst,
    unsigned xmm_src,
    unsigned storage,
-   unsigned char *code )
+   void (*code)() )
 {
    x86_push(
       func,
@@ -506,7 +506,7 @@ emit_cos(
       func,
       xmm_dst,
       ALIGN16( (unsigned) g_cos_storage ),
-      (unsigned char *) cos4f );
+      cos4f );
 }
 
 /* XXX: move into machine context */
@@ -538,7 +538,7 @@ emit_sin (struct x86_function *func,
       func,
       xmm_dst,
       ALIGN16( (unsigned) g_sin_storage ),
-      (unsigned char *) sin4f );
+      sin4f );
 }
 
 static void
@@ -596,7 +596,7 @@ emit_pow(
       xmm_dst,
       xmm_src,
       ALIGN16( (unsigned) g_pow_storage ),
-      (unsigned char *) pow4f );
+      pow4f );
 }
 
 /* XXX: move into machine context */
@@ -629,7 +629,7 @@ emit_ex2(
       func,
       xmm_dst,
       ALIGN16( (unsigned) g_ex2_storage ),
-      (unsigned char *) ex24f );
+      ex24f );
 }
 
 /* XXX: move into machine context */
@@ -655,7 +655,7 @@ emit_lg2(
       func,
       xmm_dst,
       ALIGN16( (unsigned) g_lg2_storage ),
-      (unsigned char *) lg24f );
+      lg24f );
 }
 
 /* XXX: move into machine context */
@@ -681,7 +681,7 @@ emit_flr(
       func,
       xmm_dst,
       ALIGN16( (unsigned) g_flr_storage ),
-      (unsigned char *) flr4f );
+      flr4f );
 }
 
 /* XXX: move into machine context */
@@ -707,7 +707,7 @@ emit_frc(
       func,
       xmm_dst,
       ALIGN16( (unsigned) g_frc_storage ),
-      (unsigned char *) frc4f );
+      frc4f );
 }
 
 static void

diff --git a/src/mesa/x86/rtasm/x86sse.c b/src/mesa/x86/rtasm/x86sse.c
index 96c1301d7f5b3d0b6737b6ea0c99988fbf0d61da..a6cfa40f91f7d2f83f543b888d95e9a626e733ed 100644 (file)
--- a/src/mesa/x86/rtasm/x86sse.c
+++ b/src/mesa/x86/rtasm/x86sse.c
@@ -6,6 +6,12 @@
 #define DISASSEM 0
 #define X86_TWOB 0x0f
 
+static GLubyte *cptr( void (*label)() )
+{
+   return (char *)(unsigned long)label;
+}
+
+
 /* Emit bytes to the instruction stream:
  */
 static void emit_1b( struct x86_function *p, GLbyte b0 )
@@ -252,10 +258,10 @@ void x86_jmp( struct x86_function *p, GLubyte *label)
    emit_1i(p, label - x86_get_label(p) - 4);
 }
 
-void x86_call( struct x86_function *p, GLubyte *label)
+void x86_call( struct x86_function *p, void (*label)())
 {
    emit_1ub(p, 0xe8);
-   emit_1i(p, label - x86_get_label(p) - 4);
+   emit_1i(p, cptr(label) - x86_get_label(p) - 4);
 }
 
 /* michal:
@@ -1138,7 +1144,7 @@ void (*x86_get_func( struct x86_function *p ))(void)
 {
    if (DISASSEM)
       _mesa_printf("disassemble %p %p\n", p->store, p->csr);
-   return (void (*)(void))p->store;
+   return (void (*)(void)) (unsigned long) p->store;
 }
 
 #else

diff --git a/src/mesa/x86/rtasm/x86sse.h b/src/mesa/x86/rtasm/x86sse.h
index 4816bd2ad2ce1741a24cb5bfe49b35a01dd71a85..c1ca06088bcc7b89e5290bf37ec58af4d46d6c8d 100644 (file)
--- a/src/mesa/x86/rtasm/x86sse.h
+++ b/src/mesa/x86/rtasm/x86sse.h
@@ -120,7 +120,7 @@ void x86_fixup_fwd_jump( struct x86_function *p,
 
 void x86_jmp( struct x86_function *p, GLubyte *label );
 
-void x86_call( struct x86_function *p, GLubyte *label );
+void x86_call( struct x86_function *p, void (*label)() );
 
 /* michal:
  * Temporary. As I need immediate operands, and dont want to mess with the codegen,