Ivan Gyurdiev : wined3d: Share get_register_code().

Alexandre Julliard julliard at wine.codeweavers.com
Mon May 8 09:05:45 CDT 2006


Module: wine
Branch: refs/heads/master
Commit: 4b3079438b193e60d0b6af1479560b15a4cacaf0
URL:    http://source.winehq.org/git/?p=wine.git;a=commit;h=4b3079438b193e60d0b6af1479560b15a4cacaf0

Author: Ivan Gyurdiev <ivg2 at cornell.edu>
Date:   Fri Apr 28 05:20:48 2006 -0400

wined3d: Share get_register_code().

---

 dlls/wined3d/pixelshader.c     |   16 ++++------------
 dlls/wined3d/vertexshader.c    |   25 ++++++++++---------------
 dlls/wined3d/wined3d_private.h |    5 +++++
 3 files changed, 19 insertions(+), 27 deletions(-)

diff --git a/dlls/wined3d/pixelshader.c b/dlls/wined3d/pixelshader.c
index 323fdf2..e4397e8 100644
--- a/dlls/wined3d/pixelshader.c
+++ b/dlls/wined3d/pixelshader.c
@@ -752,7 +752,7 @@ inline static void get_register_name(con
     static const char* rastout_reg_names[] = { "oC0", "oC1", "oC2", "oC3", "oDepth" };
 
     DWORD reg = param & D3DSP_REGNUM_MASK;
-    DWORD regtype = ((param & D3DSP_REGTYPE_MASK) >> D3DSP_REGTYPE_SHIFT);
+    DWORD regtype = shader_get_regtype(param);
 
     switch (regtype) {
     case D3DSPR_TEMP:
@@ -978,7 +978,7 @@ inline static void pshader_program_get_r
             int i;
                         
             for (i = 0; i < curOpcode->num_params; ++i) {
-                DWORD regtype = (((*pToken) & D3DSP_REGTYPE_MASK) >> D3DSP_REGTYPE_SHIFT);
+                DWORD regtype = shader_get_regtype(*pToken);
                 DWORD reg = (*pToken) & D3DSP_REGNUM_MASK;
                 if (D3DSPR_TEXTURE == regtype) 
                     *texUsed |= (1 << reg);
@@ -1614,14 +1614,8 @@ inline static void pshader_program_dump_
   static const char* rastout_reg_names[] = { "oC0", "oC1", "oC2", "oC3", "oDepth" };
   static const char swizzle_reg_chars[] = "rgba";
 
-   /* the unknown mask is for bits not yet accounted for by any other mask... */
-#define UNKNOWN_MASK 0xC000
-
-   /* for registeres about 7 we have to add on bits 11 and 12 to get the correct register */
-#define EXTENDED_REG 0x1800
-
   DWORD reg = param & D3DSP_REGNUM_MASK;
-  DWORD regtype = ((param & D3DSP_REGTYPE_MASK) >> D3DSP_REGTYPE_SHIFT) | ((param & EXTENDED_REG) >> 8);
+  DWORD regtype = shader_get_regtype(param);
 
   if (input) {
     if ( ((param & D3DSP_SRCMOD_MASK) == D3DSPSM_NEG) ||
@@ -1737,9 +1731,7 @@ #define EXTENDED_REG 0x1800
 inline static void pshader_program_dump_decl_usage(
     IWineD3DPixelShaderImpl *This, DWORD decl, DWORD param) {
 
-    DWORD regtype = ((param & D3DSP_REGTYPE_MASK) >> D3DSP_REGTYPE_SHIFT) | 
-                    ((param & D3DSP_REGTYPE_MASK2) >> D3DSP_REGTYPE_SHIFT2);
-
+    DWORD regtype = shader_get_regtype(param);
     TRACE("dcl_");
 
     if (regtype == D3DSPR_SAMPLER) {
diff --git a/dlls/wined3d/vertexshader.c b/dlls/wined3d/vertexshader.c
index a16532f..d211ea7 100644
--- a/dlls/wined3d/vertexshader.c
+++ b/dlls/wined3d/vertexshader.c
@@ -569,17 +569,12 @@ inline static const SHADER_OPCODE* vshad
     return NULL;
 }
 
-inline static int vshader_program_get_regtype(const DWORD param) {
-    return (((param & D3DSP_REGTYPE_MASK) >> D3DSP_REGTYPE_SHIFT) |
-                    ((param & D3DSP_REGTYPE_MASK2) >> D3DSP_REGTYPE_SHIFT2));
-}
-
 inline static void vshader_program_dump_param(const DWORD param, int input) {
   static const char* rastout_reg_names[] = { "oPos", "oFog", "oPts" };
   static const char swizzle_reg_chars[] = "xyzw";
 
   DWORD reg = param & D3DSP_REGNUM_MASK;
-  DWORD regtype = vshader_program_get_regtype(param);
+  DWORD regtype = shader_get_regtype(param);
 
   if ((param & D3DSP_SRCMOD_MASK) == D3DSPSM_NEG) TRACE("-");
 
@@ -650,11 +645,11 @@ inline static void vshader_program_dump_
   static const char swizzle_reg_chars[] = "xyzw";
 
   DWORD reg = param & D3DSP_REGNUM_MASK; 
-  DWORD regtype = vshader_program_get_regtype(param);
+  DWORD regtype = shader_get_regtype(param);
 
   if ((param & D3DSP_SRCMOD_MASK) == D3DSPSM_NEG) TRACE("-");
 
-  switch (regtype /*<< D3DSP_REGTYPE_SHIFT*/) {
+  switch (regtype) {
   case D3DSPR_TEMP:
     TRACE("r%lu", reg);
     break;
@@ -733,7 +728,7 @@ inline static void vshader_program_dump_
 
 inline static void vshader_program_dump_decl_usage(
     IWineD3DVertexShaderImpl *This, DWORD decl, DWORD param) {
-    DWORD regtype = vshader_program_get_regtype(param);
+    DWORD regtype = shader_get_regtype(param);
 
     TRACE("dcl_");
 
@@ -879,7 +874,7 @@ inline static void vshader_program_add_p
   static const char* hwrastout_reg_names[] = { "result.position", "result.fogcoord", "result.pointsize" };
 
   DWORD reg = param & D3DSP_REGNUM_MASK;
-  DWORD regtype = vshader_program_get_regtype(param);
+  DWORD regtype = shader_get_regtype(param);
   char  tmpReg[255];
   BOOL is_color = FALSE;
 
@@ -1218,7 +1213,7 @@ #define PNSTRCAT(_pgmStr, _tmpLine) { \
                 } else {
                     /* Check to see if and tmp or addressing redisters are used */
                     if (curOpcode->num_params > 0) {
-                        regtype = vshader_program_get_regtype(*pToken);
+                        regtype = shader_get_regtype(*pToken);
                         reg = ((*pToken)  & D3DSP_REGNUM_MASK);
                         if (D3DSPR_ADDR == regtype && nUseAddressRegister <= reg) nUseAddressRegister = reg + 1;
                         if (D3DSPR_TEMP == regtype){
@@ -1227,7 +1222,7 @@ #define PNSTRCAT(_pgmStr, _tmpLine) { \
                         }
                         ++pToken;
                         for (i = 1; i < curOpcode->num_params; ++i) {
-                            regtype = vshader_program_get_regtype(*pToken);
+                            regtype = shader_get_regtype(*pToken);
                             reg = ((*pToken)  & D3DSP_REGNUM_MASK);
                             if (D3DSPR_ADDR == regtype && nUseAddressRegister <= reg) nUseAddressRegister = reg + 1;
                             if (D3DSPR_TEMP == regtype){
@@ -1501,7 +1496,7 @@ #endif
             continue;
         case D3DSIO_MOV:
             /* Address registers must be loaded with the ARL instruction */
-            if (vshader_program_get_regtype(*pToken) == D3DSPR_ADDR) {
+            if (shader_get_regtype(*pToken) == D3DSPR_ADDR) {
                 if (((*pToken) & D3DSP_REGNUM_MASK) < nUseAddressRegister) {
                     strcpy(tmpLine, "ARL");
                     break;
@@ -1696,9 +1691,9 @@ #endif
                 /* TRACE(">> execting opcode: pos=%d opcode_name=%s token=%08lX\n", pToken - vshader->function, curOpcode->name, *pToken); */
                 for (i = 0; i < curOpcode->num_params; ++i) {
                     DWORD reg = pToken[i] & D3DSP_REGNUM_MASK;
-                    DWORD regtype = vshader_program_get_regtype(pToken[i]);
+                    DWORD regtype = shader_get_regtype(pToken[i]);
     
-                    switch (regtype /*<< D3DSP_REGTYPE_SHIFT*/) {
+                    switch (regtype) {
                     case D3DSPR_TEMP:
                         /* TRACE("p[%d]=R[%d]\n", i, reg); */
                         p[i] = &R[reg];
diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h
index 72c30c2..0150820 100644
--- a/dlls/wined3d/wined3d_private.h
+++ b/dlls/wined3d/wined3d_private.h
@@ -1203,6 +1203,11 @@ typedef struct IWineD3DBaseShaderImpl {
     IWineD3DBaseShaderClass         baseShader;
 } IWineD3DBaseShaderImpl;
 
+inline static int shader_get_regtype(const DWORD param) {
+    return (((param & D3DSP_REGTYPE_MASK) >> D3DSP_REGTYPE_SHIFT) |
+            ((param & D3DSP_REGTYPE_MASK2) >> D3DSP_REGTYPE_SHIFT2));
+}
+
 /*****************************************************************************
  * IDirect3DVertexShader implementation structure
  */




More information about the wine-cvs mailing list