Module: Mesa
Branch: main
Commit: 7a37a39fe03f7ac95ffa049ad99d494ca13c67a1
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7a37a39fe03f7ac95ffa049ad99d494ca13c67a1

Author: Rhys Perry <[email protected]>
Date:   Mon Dec  4 14:23:05 2023 +0000

aco: simplify v_mul_* labelling slightly

This was from before VALU_instruction existed.

Signed-off-by: Rhys Perry <[email protected]>
Reviewed-by: Georg Lehmann <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26445>

---

 src/amd/compiler/aco_optimizer.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/amd/compiler/aco_optimizer.cpp 
b/src/amd/compiler/aco_optimizer.cpp
index cd8e8bf787e..2f4efa0b0aa 100644
--- a/src/amd/compiler/aco_optimizer.cpp
+++ b/src/amd/compiler/aco_optimizer.cpp
@@ -1889,15 +1889,15 @@ label_instruction(opt_ctx& ctx, aco_ptr<Instruction>& 
instr)
                  instr->operands[!i].constantEquals(fp16 ? 0xbc00 : 
0xbf800000u))) { /* -1.0 */
                bool neg1 = instr->operands[!i].constantEquals(fp16 ? 0xbc00 : 
0xbf800000u);
 
-               VALU_instruction* vop3 = instr->isVOP3() ? &instr->valu() : 
NULL;
-               if (vop3 && (vop3->abs[!i] || vop3->neg[!i] || vop3->omod))
+               VALU_instruction* valu = &instr->valu();
+               if (valu->abs[!i] || valu->neg[!i] || valu->omod)
                   continue;
 
-               bool abs = vop3 && vop3->abs[i];
-               bool neg = neg1 ^ (vop3 && vop3->neg[i]);
+               bool abs = valu->abs[i];
+               bool neg = neg1 ^ valu->neg[i];
                Temp other = instr->operands[i].getTemp();
 
-               if (vop3 && vop3->clamp) {
+               if (valu->clamp) {
                   if (!abs && !neg && other.type() == RegType::vgpr)
                      ctx.info[other.id()].set_clamp(instr.get());
                   continue;

Reply via email to