x86: emit absolute calls, as reallocating exec mem breaks relative ones
authorKeith Whitwell <keith@tungstengraphics.com>
Wed, 13 Feb 2008 12:35:16 +0000 (12:35 +0000)
committerKeith Whitwell <keith@tungstengraphics.com>
Wed, 13 Feb 2008 12:36:32 +0000 (12:36 +0000)
src/mesa/pipe/tgsi/exec/tgsi_sse2.c
src/mesa/x86/rtasm/x86sse.c
src/mesa/x86/rtasm/x86sse.h

index ecf4ca8d21eba62f19e58a78fbe0a1c7e2deb9d1..1e56e4afb69c02843d45b8ea83eee056eab4d6af 100755 (executable)
@@ -328,8 +328,11 @@ emit_call(
    struct x86_function  *func,
    void                 (* addr)() )
 {
+   struct x86_reg ecx = x86_make_reg( file_REG32, reg_CX );
+
    DUMP_I( "CALL", addr );
-   x86_call( func, addr );
+   x86_mov_reg_imm( func, ecx, (unsigned long) addr );
+   x86_call( func, ecx );
 }
 
 static void
index 385fb84c01f8d7eae448ae76f22ff5fbba33440b..e944d00f9e842147c762bc1533ba397a3b7c6201 100644 (file)
@@ -278,11 +278,24 @@ void x86_jmp( struct x86_function *p, unsigned char *label)
    emit_1i(p, label - x86_get_label(p) - 4);
 }
 
+#if 0
+/* This doesn't work once we start reallocating & copying the
+ * generated code on buffer fills, because the call is relative to the
+ * current pc.
+ */
 void x86_call( struct x86_function *p, void (*label)())
 {
    emit_1ub(p, 0xe8);
    emit_1i(p, cptr(label) - x86_get_label(p) - 4);
 }
+#else
+void x86_call( struct x86_function *p, struct x86_reg reg)
+{
+   emit_1ub(p, 0xff);
+   emit_modrm(p, reg, reg);
+}
+#endif
+
 
 /* michal:
  * Temporary. As I need immediate operands, and dont want to mess with the codegen,
index d53b6d71a6be07271451911ef58d053a5449046a..c2aa416492ee54eade74a23ef4ea6ae58b63fb7c 100644 (file)
@@ -119,7 +119,8 @@ void x86_fixup_fwd_jump( struct x86_function *p,
 
 void x86_jmp( struct x86_function *p, unsigned char *label );
 
-void x86_call( struct x86_function *p, void (*label)() );
+/* void x86_call( struct x86_function *p, void (*label)() ); */
+void x86_call( struct x86_function *p, struct x86_reg reg);
 
 /* michal:
  * Temporary. As I need immediate operands, and dont want to mess with the codegen,