From: Iago Toral Quiroga <itoral@igalia.com>
Date: Mon, 21 Jan 2019 08:47:59 +0000 (+0100)
Subject: intel/compiler: drop unnecessary temporary from 32-bit fsign implementation
X-Git-Tag: upstream/19.3.0~7233
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a8d8b1a1391b207d4b19b1ba864612837f1fd543;p=platform%2Fupstream%2Fmesa.git

intel/compiler: drop unnecessary temporary from 32-bit fsign implementation

Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
---

diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp
index dbc756f..0a4ccc5 100644
--- a/src/intel/compiler/brw_fs_nir.cpp
+++ b/src/intel/compiler/brw_fs_nir.cpp
@@ -972,12 +972,11 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr)
           */
          bld.CMP(bld.null_reg_f(), op[0], brw_imm_f(0.0f), BRW_CONDITIONAL_NZ);
 
-         fs_reg result_int = retype(result, BRW_REGISTER_TYPE_UD);
          op[0].type = BRW_REGISTER_TYPE_UD;
          result.type = BRW_REGISTER_TYPE_UD;
-         bld.AND(result_int, op[0], brw_imm_ud(0x80000000u));
+         bld.AND(result, op[0], brw_imm_ud(0x80000000u));
 
-         inst = bld.OR(result_int, result_int, brw_imm_ud(0x3f800000u));
+         inst = bld.OR(result, result, brw_imm_ud(0x3f800000u));
          inst->predicate = BRW_PREDICATE_NORMAL;
       } else {
          /* For doubles we do the same but we need to consider: