diff options
| author | Matt Turner <[email protected]> | 2014-05-02 14:47:55 -0700 |
|---|---|---|
| committer | Matt Turner <[email protected]> | 2014-05-24 23:03:24 -0700 |
| commit | 99af02fb1778a425f1f3d211018d294b8d3b2245 (patch) | |
| tree | c1036285bd57f81f73e4f6aaf92fc10cbee2bed5 /src | |
| parent | fb977c90d1ef29f47b686c27500005025543cf11 (diff) | |
i965: Emit 0.0:F sources with type VF instead.
Number of compacted instructions: 817752 -> 827404 (1.18%)
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_eu_emit.c | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index d8efa017386..1810233c143 100644 --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c @@ -357,6 +357,22 @@ brw_set_src0(struct brw_compile *p, struct brw_instruction *insn, } else { insn->bits1.da1.src1_reg_type = BRW_HW_REG_TYPE_UD; } + + /* Compacted instructions only have 12-bits (plus 1 for the other 20) + * for immediate values. Presumably the hardware engineers realized + * that the only useful floating-point value that could be represented + * in this format is 0.0, which can also be represented as a VF-typed + * immediate, so they gave us the previously mentioned mapping on IVB+. + * + * Strangely, we do have a mapping for imm:f in src1, so we don't need + * to do this there. + * + * If we see a 0.0:F, change the type to VF so that it can be compacted. + */ + if (insn->bits3.ud == 0x0 && + insn->bits1.da1.src0_reg_type == BRW_HW_REG_TYPE_F) { + insn->bits1.da1.src0_reg_type = BRW_HW_REG_IMM_TYPE_VF; + } } else { |