[PATCH 1/5] d3dx9: Avoid casting each value separately in set_constants().

[Paul Gofman]

Signed-off-by: Paul Gofman <gofmanp at gmail.com>
---
 dlls/d3dx9_36/preshader.c | 98 +++++++++++++++++++++++++++++++++++++++--------
 1 file changed, 81 insertions(+), 17 deletions(-)

diff --git a/dlls/d3dx9_36/preshader.c b/dlls/d3dx9_36/preshader.c
index 522d484..f73e4a4 100644
--- a/dlls/d3dx9_36/preshader.c
+++ b/dlls/d3dx9_36/preshader.c
@@ -1032,6 +1032,79 @@ static void get_const_upload_info(struct d3dx_const_param_eval_output *const_set
     info->count = info->major_count * info->minor;
 }
 
+static void pres_float_to_int(void *in, void *out, unsigned int count)
+{
+    unsigned int i;
+    const float *in_float = in;
+    int *out_int = out;
+
+    for (i = 0; i < count; ++i)
+        out_int[i] = in_float[i];
+}
+
+static void pres_value_to_bool(void *in, void *out, unsigned int count)
+{
+    unsigned int i;
+    const DWORD *in_dword = in;
+    BOOL *out_bool = out;
+
+    for (i = 0; i < count; ++i)
+        out_bool[i] = !!in_dword[i];
+}
+
+static void pres_int_to_float(void *in, void *out, unsigned int count)
+{
+    unsigned int i;
+    const int *in_int = in;
+    float *out_float = out;
+
+    for (i = 0; i < count; ++i)
+        out_float[i] = in_int[i];
+}
+
+static void pres_bool_to_float(void *in, void *out, unsigned int count)
+{
+    unsigned int i;
+    const BOOL *in_bool = in;
+    float *out_float = out;
+
+    for (i = 0; i < count; ++i)
+        out_float[i] = !!in_bool[i];
+}
+
+static void pres_bool_to_int(void *in, void * out, unsigned int count)
+{
+    unsigned int i;
+    const float *in_bool = in;
+    int *out_int = out;
+
+    for (i = 0; i < count; ++i)
+        out_int[i] = !!in_bool[i];
+}
+
+static void regstore_set_data(struct d3dx_regstore *rs, unsigned int table,
+        unsigned int offset, unsigned int *in, unsigned int count, enum pres_value_type param_type)
+{
+    typedef void (*conv_func)(void *in, void *out, unsigned int count);
+    static conv_func set_const_funcs[PRES_VT_COUNT][PRES_VT_COUNT] =
+    {
+        {NULL,               NULL, pres_float_to_int, pres_value_to_bool},
+        {NULL,               NULL, NULL,              NULL},
+        {pres_int_to_float,  NULL, NULL,              pres_value_to_bool},
+        {pres_bool_to_float, NULL, pres_bool_to_int,  NULL}
+    };
+    enum pres_value_type table_type = table_info[table].type;
+
+    if (table_type == param_type)
+    {
+        regstore_set_values(rs, table, in, offset, count);
+        return;
+    }
+
+    set_const_funcs[param_type][table_type](in, (unsigned int *)rs->tables[table] + offset, count);
+    regstore_set_modified(rs, table, offset, count);
+}
+
 static void set_constants(struct d3dx_regstore *rs, struct d3dx_const_tab *const_tab,
         ULONG64 new_update_version)
 {
@@ -1042,11 +1115,11 @@ static void set_constants(struct d3dx_regstore *rs, struct d3dx_const_tab *const
         struct d3dx_const_param_eval_output *const_set = &const_tab->const_set[const_idx];
         unsigned int table = const_set->table;
         struct d3dx_parameter *param = const_set->param;
-        enum pres_value_type table_type = table_info[table].type;
         unsigned int element, i, j, start_offset;
         unsigned int param_offset;
         struct const_upload_info info;
         unsigned int *data = param->data;
+        enum pres_value_type param_type;
 
         if (!is_param_dirty(param, const_tab->update_version))
             continue;
@@ -1058,22 +1131,21 @@ static void set_constants(struct d3dx_regstore *rs, struct d3dx_const_tab *const
                     get_offset_reg(table, const_set->register_count));
             continue;
         }
+        param_type = table_type_from_param_type(param->type);
         get_const_upload_info(const_set, &info);
-
         for (element = 0; element < const_set->element_count; ++element)
         {
+            unsigned int *out = (unsigned int *)rs->tables[table] + start_offset;
+
             for (i = 0; i < info.major_count; ++i)
             {
                 for (j = 0; j < info.minor; ++j)
                 {
-                    unsigned int out;
-                    unsigned int *in;
                     unsigned int offset;
 
                     offset = i * info.major_stride + j;
                     if (get_reg_offset(table, offset) >= const_set->register_count)
                         break;
-                    offset += start_offset;
                     if (info.transpose)
                         param_offset = i + j * info.major;
                     else
@@ -1083,23 +1155,15 @@ static void set_constants(struct d3dx_regstore *rs, struct d3dx_const_tab *const
                         WARN("Parameter data is too short, name %s, component %u.\n", debugstr_a(param->name), i);
                         break;
                     }
-
-                    in = (unsigned int *)data + param_offset;
-                    switch (table_type)
-                    {
-                        case PRES_VT_FLOAT: set_number(&out, D3DXPT_FLOAT, in, param->type); break;
-                        case PRES_VT_INT: set_number(&out, D3DXPT_INT, in, param->type); break;
-                        case PRES_VT_BOOL: set_number(&out, D3DXPT_BOOL, in, param->type); break;
-                        default:
-                            FIXME("Unexpected type %#x.\n", table_info[table].type);
-                            break;
-                    }
-                    regstore_set_values(rs, table, &out, offset, 1);
+                    out[offset] = data[param_offset];
                 }
             }
             start_offset += get_offset_reg(table, const_set->register_count);
             data += param->rows * param->columns;
         }
+        start_offset = get_offset_reg(table, const_set->register_index);
+        regstore_set_data(rs, table, start_offset, (unsigned int *)rs->tables[table] + start_offset,
+                get_offset_reg(table, const_set->register_count) * const_set->element_count, param_type);
     }
     const_tab->update_version = new_update_version;
 }
-- 
2.9.4