diff options
| author | Chris Forbes <[email protected]> | 2014-08-04 19:41:03 +1200 |
|---|---|---|
| committer | Chris Forbes <[email protected]> | 2014-08-09 13:12:31 +1200 |
| commit | 6be68767b9b5344d5753b8909f5ec8f57309b71a (patch) | |
| tree | a335a85eda017be71652d99ec373c4ea302ebe51 /src/mesa/drivers | |
| parent | 1a3fd11aefdf6ed327f633ea7e13bae2e8a92ca7 (diff) | |
i965/vec4/Gen4-7: Use src1 for sampler_index instead of ->sampler field
Signed-off-by: Chris Forbes <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/mesa/drivers')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4.h | 5 | ||||
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 18 |
2 files changed, 15 insertions, 8 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h b/src/mesa/drivers/dri/i965/brw_vec4.h index 74a0568e0c6..7c2721dec1e 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.h +++ b/src/mesa/drivers/dri/i965/brw_vec4.h @@ -644,8 +644,9 @@ private: struct brw_reg src1); void generate_tex(vec4_instruction *inst, - struct brw_reg dst, - struct brw_reg src); + struct brw_reg dst, + struct brw_reg src, + struct brw_reg sampler_index); void generate_vs_urb_write(vec4_instruction *inst); void generate_gs_urb_write(vec4_instruction *inst); diff --git a/src/mesa/drivers/dri/i965/brw_vec4_generator.cpp b/src/mesa/drivers/dri/i965/brw_vec4_generator.cpp index 2129007f988..f362221ff49 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_generator.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_generator.cpp @@ -225,7 +225,8 @@ vec4_generator::generate_math2_gen4(vec4_instruction *inst, void vec4_generator::generate_tex(vec4_instruction *inst, struct brw_reg dst, - struct brw_reg src) + struct brw_reg src, + struct brw_reg sampler_index) { int msg_type = -1; @@ -313,6 +314,11 @@ vec4_generator::generate_tex(vec4_instruction *inst, assert(msg_type != -1); + assert(sampler_index.file == BRW_IMMEDIATE_VALUE); + assert(sampler_index.type == BRW_REGISTER_TYPE_UD); + + uint32_t sampler = sampler_index.dw1.ud; + /* Load the message header if present. If there's a texture offset, we need * to set it up explicitly and load the offset bitfield. Otherwise, we can * use an implied move from g0 to the first message register. @@ -338,7 +344,7 @@ vec4_generator::generate_tex(vec4_instruction *inst, brw_imm_ud(inst->texture_offset)); } - if (inst->sampler >= 16) { + if (sampler >= 16) { /* The "Sampler Index" field can only store values between 0 and 15. * However, we can add an offset to the "Sampler State Pointer" * field, effectively selecting a different set of 16 samplers. @@ -352,7 +358,7 @@ vec4_generator::generate_tex(vec4_instruction *inst, brw_ADD(p, get_element_ud(header, 3), get_element_ud(brw_vec8_grf(0, 0), 3), - brw_imm_ud(16 * (inst->sampler / 16) * sampler_state_size)); + brw_imm_ud(16 * (sampler / 16) * sampler_state_size)); } brw_pop_insn_state(p); } @@ -375,14 +381,14 @@ vec4_generator::generate_tex(vec4_instruction *inst, uint32_t surface_index = ((inst->opcode == SHADER_OPCODE_TG4 || inst->opcode == SHADER_OPCODE_TG4_OFFSET) ? prog_data->base.binding_table.gather_texture_start - : prog_data->base.binding_table.texture_start) + inst->sampler; + : prog_data->base.binding_table.texture_start) + sampler; brw_SAMPLE(p, dst, inst->base_mrf, src, surface_index, - inst->sampler % 16, + sampler % 16, msg_type, 1, /* response length */ inst->mlen, @@ -1131,7 +1137,7 @@ vec4_generator::generate_vec4_instruction(vec4_instruction *instruction, case SHADER_OPCODE_TXS: case SHADER_OPCODE_TG4: case SHADER_OPCODE_TG4_OFFSET: - generate_tex(inst, dst, src[0]); + generate_tex(inst, dst, src[0], src[1]); break; case VS_OPCODE_URB_WRITE: |