diff options
| author | Matt Turner <[email protected]> | 2015-02-12 02:01:49 +0000 |
|---|---|---|
| committer | Matt Turner <[email protected]> | 2015-02-15 12:24:10 -0800 |
| commit | 72b9f8db2a035b52dbd59af0d2a6441cbde11a4c (patch) | |
| tree | c8eabdd78a0c911a4c0de8316f3e9f7a88ee3994 /src | |
| parent | 69b1693ef3e5f6be872ae131ea01de435a8e9337 (diff) | |
i965/vec4/vp: Use vec4_visitor::CMP.
... instead of emit(BRW_OPCODE_CMP, ...). In commit 6b3a301f I changed
vec4_visitor::CMP to set the destination's type to that of src0. In the
following commit (2335153f) I removed an apparently now unnecessary work
around for Gen8 that did the same thing.
But there was a single place that emitted a CMP instruction without
using the vec4_visitor::CMP function. Use it there.
And change dst_null_d to dst_null_f for good measure, since ARB vp
doesn't have integers.
Cc: "10.5" <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89032
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4_vp.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_vp.cpp b/src/mesa/drivers/dri/i965/brw_vec4_vp.cpp index edd92ed3680..60a0f24154a 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_vp.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_vp.cpp @@ -43,8 +43,7 @@ vec4_visitor::emit_vp_sop(enum brw_conditional_mod conditional_mod, { vec4_instruction *inst; - inst = emit(BRW_OPCODE_CMP, dst_null_d(), src0, src1); - inst->conditional_mod = conditional_mod; + inst = emit(CMP(dst_null_f(), src0, src1, conditional_mod)); inst = emit(BRW_OPCODE_SEL, dst, one, src_reg(0.0f)); inst->predicate = BRW_PREDICATE_NORMAL; |