Thanks, pushed.
On Wed, Oct 25, 2017 at 3:20 PM, Matthew Nicholls
<mnicho...@feralinteractive.com> wrote:
> v2: fix silly typo
>
> Cc: "17.2 17.3" <mesa-sta...@lists.freedesktop.org>
> ---
> src/amd/common/ac_nir_to_llvm.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
> index 3d635d4206..06937d684b 100644
> --- a/src/amd/common/ac_nir_to_llvm.c
> +++ b/src/amd/common/ac_nir_to_llvm.c
> @@ -3667,15 +3667,17 @@ static LLVMValueRef visit_image_atomic(struct
> ac_nir_context *ctx,
> LLVMValueRef i1true = LLVMConstInt(ctx->ac.i1, 1, false);
> MAYBE_UNUSED int length;
>
> + bool is_unsigned = glsl_get_sampler_result_type(type) ==
> GLSL_TYPE_UINT;
> +
> switch (instr->intrinsic) {
> case nir_intrinsic_image_atomic_add:
> atomic_name = "add";
> break;
> case nir_intrinsic_image_atomic_min:
> - atomic_name = "smin";
> + atomic_name = is_unsigned ? "umin" : "smin";
> break;
> case nir_intrinsic_image_atomic_max:
> - atomic_name = "smax";
> + atomic_name = is_unsigned ? "umax" : "smax";
> break;
> case nir_intrinsic_image_atomic_and:
> atomic_name = "and";
> --
> 2.13.6
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
