fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-28 08:10:09 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

ac/nir: do not sign-extend the result of texop_samples_identical

Browse source 
The return value should be a NIR 1-bit bool.

This fixes a regression with piglit
ext_shader_samples_identical-simple-fs 2 on RadeonSI.

Fixes: e690a1b78b ("ac/llvm: don't lower bool to int32, switch to native i1 bool")
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7236>
This commit is contained in:
Samuel Pitoiset 2020-10-20 16:54:45 +02:00
parent 11d70e7ab9
commit eacdad7ea0
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/amd/llvm/ac_nir_to_llvm.c
View file

@ -4254,9 +4254,7 @@ static void visit_tex(struct ac_nir_context *ctx, nir_tex_instr *instr)
result = build_tex_intrinsic(ctx, instr, &txf_args);
result = LLVMBuildExtractElement(ctx->ac.builder, result, ctx->ac.i32_0, "");
result = LLVMBuildSExt(ctx->ac.builder,
emit_int_cmp(&ctx->ac, LLVMIntEQ, result, ctx->ac.i32_0),
ctx->ac.i32, "");
result = emit_int_cmp(&ctx->ac, LLVMIntEQ, result, ctx->ac.i32_0);
goto write_result;
}