[Mesa-dev] [PATCH 28/42] i965/blorp: wrap MOV (/brw_MOV(&func, /emit_mov(/)
Topi Pohjolainen
topi.pohjolainen at intel.com
Fri Dec 20 06:38:57 PST 2013
In addition, the two special cases requiring explicit execution
size control are wrapped manually.
Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
---
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 68 +++++++++++++--------------
src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h | 12 +++++
2 files changed, 45 insertions(+), 35 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
index b95104e..cedf580 100644
--- a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
+++ b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
@@ -1041,8 +1041,8 @@ brw_blorp_blit_program::compute_frag_coords()
stride(suboffset(R1, 5), 2, 4, 0), brw_imm_v(0x11001100));
/* Move the coordinates to UD registers. */
- brw_MOV(&func, vec16(Xp), retype(X, BRW_REGISTER_TYPE_UW));
- brw_MOV(&func, vec16(Yp), retype(Y, BRW_REGISTER_TYPE_UW));
+ emit_mov(vec16(Xp), retype(X, BRW_REGISTER_TYPE_UW));
+ emit_mov(vec16(Yp), retype(Y, BRW_REGISTER_TYPE_UW));
SWAP_XY_AND_XPYP();
if (key->persample_msaa_dispatch) {
@@ -1058,12 +1058,10 @@ brw_blorp_blit_program::compute_frag_coords()
* then copy from it using vstride=1, width=4, hstride=0.
*/
struct brw_reg t1_uw1 = retype(t1, BRW_REGISTER_TYPE_UW);
- brw_MOV(&func, vec16(t1_uw1), brw_imm_v(0x3210));
+ emit_mov(vec16(t1_uw1), brw_imm_v(0x3210));
/* Move to UD sample_index register. */
- brw_set_compression_control(&func, BRW_COMPRESSION_NONE);
- brw_MOV(&func, S, stride(t1_uw1, 1, 4, 0));
- brw_MOV(&func, offset(S, 1), suboffset(stride(t1_uw1, 1, 4, 0), 2));
- brw_set_compression_control(&func, BRW_COMPRESSION_COMPRESSED);
+ emit_mov_8(S, stride(t1_uw1, 1, 4, 0));
+ emit_mov_8(offset(S, 1), suboffset(stride(t1_uw1, 1, 4, 0), 2));
break;
}
case 8: {
@@ -1085,7 +1083,7 @@ brw_blorp_blit_program::compute_frag_coords()
struct brw_reg r0_ud1 = vec1(retype(R0, BRW_REGISTER_TYPE_UD));
brw_AND(&func, t1_ud1, r0_ud1, brw_imm_ud(0xc0));
brw_SHR(&func, t1_ud1, t1_ud1, brw_imm_ud(5));
- brw_MOV(&func, vec16(t2_uw1), brw_imm_v(0x3210));
+ emit_mov(vec16(t2_uw1), brw_imm_v(0x3210));
brw_ADD(&func, vec16(S), retype(t1_ud1, BRW_REGISTER_TYPE_UW),
stride(t2_uw1, 1, 4, 0));
brw_set_compression_control(&func, BRW_COMPRESSION_NONE);
@@ -1383,8 +1381,8 @@ brw_blorp_blit_program::translate_dst_to_src()
struct brw_reg Yp_f = retype(Yp, BRW_REGISTER_TYPE_F);
/* Move the UD coordinates to float registers. */
- brw_MOV(&func, Xp_f, X);
- brw_MOV(&func, Yp_f, Y);
+ emit_mov(Xp_f, X);
+ emit_mov(Yp_f, Y);
/* Scale and offset */
brw_MUL(&func, X_f, Xp_f, x_transform.multiplier);
brw_MUL(&func, Y_f, Yp_f, y_transform.multiplier);
@@ -1425,8 +1423,8 @@ brw_blorp_blit_program::translate_dst_to_src()
/* Round the float coordinates down to nearest integer by moving to
* UD registers.
*/
- brw_MOV(&func, Xp, X_f);
- brw_MOV(&func, Yp, Y_f);
+ emit_mov(Xp, X_f);
+ emit_mov(Yp, Y_f);
SWAP_XY_AND_XPYP();
}
}
@@ -1528,7 +1526,7 @@ brw_blorp_blit_program::manual_blend_average(unsigned num_samples)
s_is_zero = true;
} else {
s_is_zero = false;
- brw_MOV(&func, vec16(S), brw_imm_ud(i));
+ emit_mov(vec16(S), brw_imm_ud(i));
}
texel_fetch(texture_data[stack_depth++]);
@@ -1612,8 +1610,8 @@ brw_blorp_blit_program::manual_blend_bilinear(unsigned num_samples)
brw_imm_f((float)(i & 0x1) * (1.0 / key->x_scale)));
brw_ADD(&func, vec16(y_sample_coords), Yp_f,
brw_imm_f((float)((i >> 1) & 0x1) * (1.0 / key->y_scale)));
- brw_MOV(&func, vec16(X), x_sample_coords);
- brw_MOV(&func, vec16(Y), y_sample_coords);
+ emit_mov(vec16(X), x_sample_coords);
+ emit_mov(vec16(Y), y_sample_coords);
/* The MCS value we fetch has to match up with the pixel that we're
* sampling from. Since we sample from different pixels in each
@@ -1652,7 +1650,7 @@ brw_blorp_blit_program::manual_blend_bilinear(unsigned num_samples)
brw_MUL(&func, vec16(t1_f), t1_f, brw_imm_f(key->x_scale));
brw_MUL(&func, vec16(t2_f), t2_f, brw_imm_f(key->x_scale * key->y_scale));
brw_ADD(&func, vec16(t1_f), t1_f, t2_f);
- brw_MOV(&func, vec16(S), t1_f);
+ emit_mov(vec16(S), t1_f);
if (num_samples == 8) {
/* Map the sample index to a sample number */
@@ -1660,20 +1658,20 @@ brw_blorp_blit_program::manual_blend_bilinear(unsigned num_samples)
S, brw_imm_d(4));
brw_IF(&func, BRW_EXECUTE_16);
{
- brw_MOV(&func, vec16(t2), brw_imm_d(5));
+ emit_mov(vec16(t2), brw_imm_d(5));
emit_if_eq_mov(S, 1, vec16(t2), 2);
emit_if_eq_mov(S, 2, vec16(t2), 4);
emit_if_eq_mov(S, 3, vec16(t2), 6);
}
brw_ELSE(&func);
{
- brw_MOV(&func, vec16(t2), brw_imm_d(0));
+ emit_mov(vec16(t2), brw_imm_d(0));
emit_if_eq_mov(S, 5, vec16(t2), 3);
emit_if_eq_mov(S, 6, vec16(t2), 7);
emit_if_eq_mov(S, 7, vec16(t2), 1);
}
brw_ENDIF(&func);
- brw_MOV(&func, vec16(S), t2);
+ emit_mov(vec16(S), t2);
}
texel_fetch(texture_data[i]);
}
@@ -1806,23 +1804,23 @@ brw_blorp_blit_program::texture_lookup(struct brw_reg dst,
switch (args[arg]) {
case SAMPLER_MESSAGE_ARG_U_FLOAT:
if (key->bilinear_filter)
- brw_MOV(&func, retype(mrf, BRW_REGISTER_TYPE_F),
- retype(X, BRW_REGISTER_TYPE_F));
+ emit_mov(retype(mrf, BRW_REGISTER_TYPE_F),
+ retype(X, BRW_REGISTER_TYPE_F));
else
- brw_MOV(&func, retype(mrf, BRW_REGISTER_TYPE_F), X);
+ emit_mov(retype(mrf, BRW_REGISTER_TYPE_F), X);
break;
case SAMPLER_MESSAGE_ARG_V_FLOAT:
if (key->bilinear_filter)
- brw_MOV(&func, retype(mrf, BRW_REGISTER_TYPE_F),
- retype(Y, BRW_REGISTER_TYPE_F));
+ emit_mov(retype(mrf, BRW_REGISTER_TYPE_F),
+ retype(Y, BRW_REGISTER_TYPE_F));
else
- brw_MOV(&func, retype(mrf, BRW_REGISTER_TYPE_F), Y);
+ emit_mov(retype(mrf, BRW_REGISTER_TYPE_F), Y);
break;
case SAMPLER_MESSAGE_ARG_U_INT:
- brw_MOV(&func, mrf, X);
+ emit_mov(mrf, X);
break;
case SAMPLER_MESSAGE_ARG_V_INT:
- brw_MOV(&func, mrf, Y);
+ emit_mov(mrf, Y);
break;
case SAMPLER_MESSAGE_ARG_SI_INT:
/* Note: on Gen7, this code may be reached with s_is_zero==true
@@ -1831,14 +1829,14 @@ brw_blorp_blit_program::texture_lookup(struct brw_reg dst,
* appropriate message register.
*/
if (s_is_zero)
- brw_MOV(&func, mrf, brw_imm_ud(0));
+ emit_mov(mrf, brw_imm_ud(0));
else
- brw_MOV(&func, mrf, S);
+ emit_mov(mrf, S);
break;
case SAMPLER_MESSAGE_ARG_MCS_INT:
switch (key->tex_layout) {
case INTEL_MSAA_LAYOUT_CMS:
- brw_MOV(&func, mrf, mcs_data);
+ emit_mov(mrf, mcs_data);
break;
case INTEL_MSAA_LAYOUT_IMS:
/* When sampling from an IMS surface, MCS data is not relevant,
@@ -1854,7 +1852,7 @@ brw_blorp_blit_program::texture_lookup(struct brw_reg dst,
}
break;
case SAMPLER_MESSAGE_ARG_ZERO_INT:
- brw_MOV(&func, mrf, brw_imm_ud(0));
+ emit_mov(mrf, brw_imm_ud(0));
break;
}
mrf.nr += 2;
@@ -1886,16 +1884,16 @@ brw_blorp_blit_program::render_target_write()
bool use_header = key->use_kill;
if (use_header) {
/* Copy R0/1 to MRF */
- brw_MOV(&func, retype(mrf_rt_write, BRW_REGISTER_TYPE_UD),
- retype(R0, BRW_REGISTER_TYPE_UD));
+ emit_mov(retype(mrf_rt_write, BRW_REGISTER_TYPE_UD),
+ retype(R0, BRW_REGISTER_TYPE_UD));
mrf_offset += 2;
}
/* Copy texture data to MRFs */
for (int i = 0; i < 4; ++i) {
/* E.g. mov(16) m2.0<1>:f r2.0<8;8,1>:f { Align1, H1 } */
- brw_MOV(&func, offset(mrf_rt_write, mrf_offset),
- offset(vec8(texture_data[0]), 2*i));
+ emit_mov(offset(mrf_rt_write, mrf_offset),
+ offset(vec8(texture_data[0]), 2*i));
mrf_offset += 2;
}
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h b/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h
index 4a7a772..5441b27 100644
--- a/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h
+++ b/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h
@@ -87,6 +87,18 @@ protected:
brw_set_access_mode(&func, BRW_ALIGN_1);
}
+ inline void emit_mov(const struct brw_reg& dst, const struct brw_reg& src)
+ {
+ brw_MOV(&func, dst, src);
+ }
+
+ inline void emit_mov_8(const struct brw_reg& dst, const struct brw_reg& src)
+ {
+ brw_set_compression_control(&func, BRW_COMPRESSION_NONE);
+ emit_mov(dst, src);
+ brw_set_compression_control(&func, BRW_COMPRESSION_COMPRESSED);
+ }
+
void *mem_ctx;
struct brw_compile func;
};
--
1.8.3.1
More information about the mesa-dev
mailing list