aco/optimizer: Don't combine uniform bool s_and to s_andn2.
authorTimur Kristóf <timur.kristof@gmail.com>
Wed, 5 Feb 2020 10:19:06 +0000 (11:19 +0100)
committerMarge Bot <eric+marge@anholt.net>
Wed, 5 Feb 2020 22:53:45 +0000 (22:53 +0000)
Fixes: 8a32f57fff56b3b94f1b5589feba38016f39427c
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3714>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3714>

src/amd/compiler/aco_optimizer.cpp

index 9ef94d4f6975329e78c5a7c92ac9c5db3a59421a..0b9fccc379e540f6c47f4de6d14350afc2fe71e8 100644 (file)
@@ -1844,6 +1844,9 @@ bool combine_salu_n2(opt_ctx& ctx, aco_ptr<Instruction>& instr)
    if (instr->definitions[1].isTemp() && ctx.uses[instr->definitions[1].tempId()])
       return false;
 
+   if (instr->definitions[0].isTemp() && ctx.info[instr->definitions[0].tempId()].is_uniform_bool())
+      return false;
+
    for (unsigned i = 0; i < 2; i++) {
       Instruction *op2_instr = follow_operand(ctx, instr->operands[i]);
       if (!op2_instr || (op2_instr->opcode != aco_opcode::s_not_b32 && op2_instr->opcode != aco_opcode::s_not_b64))