commit 0e9425a1b74ed7e38656eee43f72a527404bf0b6
author Rhys Perry <pendingchaos02@gmail.com> Thu Sep 03 15:41:32 2020 +0100
committer Marge Bot <eric+marge@anholt.net> Tue Sep 08 12:17:43 2020 +0000
tree 3d7e33fce52a8881401249a9fdb7d7d2551fa8b1
parent f4210df2241556873032f741be4a4a44337ebf99

aco: fix one-off error in Operand(uint16_t)

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Fixes: 3d6f67950d9 ('aco: improve 8/16-bit constants')
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6583>

src/amd/compiler/aco_ir.h

diff --git a/src/amd/compiler/aco_ir.h b/src/amd/compiler/aco_ir.h
index d27726d..9b3f5a3 100644
--- a/src/amd/compiler/aco_ir.h
+++ b/src/amd/compiler/aco_ir.h
@@ -449,7 +449,7 @@
       if (v <= 64)
          setFixed(PhysReg{128u + v});
       else if (v >= 0xFFF0) /* [-16 .. -1] */
-         setFixed(PhysReg{192u + (0xFFFF - v)});
+         setFixed(PhysReg{(unsigned)(192 - (int16_t)v)});
       else if (v == 0x3800) /* 0.5 */
          setFixed(PhysReg{240});
       else if (v == 0xB800) /* -0.5 */