Mesa (master): i965/fs: Clean up FBH code.

Matt Turner mattst88 at kemper.freedesktop.org
Mon Nov 2 17:36:18 UTC 2015


Module: Mesa
Branch: master
Commit: 0b19f651958c3888588190c8c8a9e701173a2aa2
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0b19f651958c3888588190c8c8a9e701173a2aa2

Author: Matt Turner <mattst88 at gmail.com>
Date:   Mon Oct 26 11:35:57 2015 -0700

i965/fs: Clean up FBH code.

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

---

 src/mesa/drivers/dri/i965/brw_fs_nir.cpp |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 24ff5af..7eeff93 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
@@ -906,12 +906,11 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr)
        * from the LSB side. If FBH didn't return an error (0xFFFFFFFF), then
        * subtract the result from 31 to convert the MSB count into an LSB count.
        */
-
       bld.CMP(bld.null_reg_d(), result, fs_reg(-1), BRW_CONDITIONAL_NZ);
-      fs_reg neg_result(result);
-      neg_result.negate = true;
-      inst = bld.ADD(result, neg_result, fs_reg(31));
+
+      inst = bld.ADD(result, result, fs_reg(31));
       inst->predicate = BRW_PREDICATE_NORMAL;
+      inst->src[0].negate = true;
       break;
    }
 




More information about the mesa-commit mailing list