ac/nir: generate correct instruction for atomic min/max on unsigned images
v2: fix silly typo Cc: "17.2 17.3" <mesa-stable@lists.freedesktop.org> Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
This commit is contained in:
parent
20c77ae639
commit
27a0b24bf2
|
@ -3667,15 +3667,17 @@ static LLVMValueRef visit_image_atomic(struct ac_nir_context *ctx,
|
||||||
LLVMValueRef i1true = LLVMConstInt(ctx->ac.i1, 1, false);
|
LLVMValueRef i1true = LLVMConstInt(ctx->ac.i1, 1, false);
|
||||||
MAYBE_UNUSED int length;
|
MAYBE_UNUSED int length;
|
||||||
|
|
||||||
|
bool is_unsigned = glsl_get_sampler_result_type(type) == GLSL_TYPE_UINT;
|
||||||
|
|
||||||
switch (instr->intrinsic) {
|
switch (instr->intrinsic) {
|
||||||
case nir_intrinsic_image_atomic_add:
|
case nir_intrinsic_image_atomic_add:
|
||||||
atomic_name = "add";
|
atomic_name = "add";
|
||||||
break;
|
break;
|
||||||
case nir_intrinsic_image_atomic_min:
|
case nir_intrinsic_image_atomic_min:
|
||||||
atomic_name = "smin";
|
atomic_name = is_unsigned ? "umin" : "smin";
|
||||||
break;
|
break;
|
||||||
case nir_intrinsic_image_atomic_max:
|
case nir_intrinsic_image_atomic_max:
|
||||||
atomic_name = "smax";
|
atomic_name = is_unsigned ? "umax" : "smax";
|
||||||
break;
|
break;
|
||||||
case nir_intrinsic_image_atomic_and:
|
case nir_intrinsic_image_atomic_and:
|
||||||
atomic_name = "and";
|
atomic_name = "and";
|
||||||
|
|
Loading…
Reference in New Issue