[Mesa-dev] [PATCH 2/2] ac/nir: Fix ordering of parameters for image atomic cmpswap intrinsics

Alex Smith asmith at feralinteractive.com
Fri Jun 30 15:56:12 UTC 2017


Just compare/swap, not all the image atomics. From a quick grep it looks
like the CTS doesn't have any tests for compare/swap, but does for the
other atomics.

Alex

On 30 June 2017 at 16:45, Jason Ekstrand <jason at jlekstrand.net> wrote:

> Does this mean that image atomics are broken and radv passes the CTS
> anyway?  If so, someone should fine a bug against the CTS so we can keep
> track of the coverage hole.
>
>
>
> On June 30, 2017 3:15:57 AM Alex Smith <asmith at feralinteractive.com>
> wrote:
>
> The NIR parameters are ordered "compare, data", matching GLSL, but both
>> the image and buffer LLVM intrinsics take them the other way around.
>> This is already handled correctly for SSBO atomics.
>>
>> Signed-off-by: Alex Smith <asmith at feralinteractive.com>
>> ---
>>  src/amd/common/ac_nir_to_llvm.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/amd/common/ac_nir_to_llvm.c
>> b/src/amd/common/ac_nir_to_llvm.c
>> index 6845df8..89b7103 100644
>> --- a/src/amd/common/ac_nir_to_llvm.c
>> +++ b/src/amd/common/ac_nir_to_llvm.c
>> @@ -3442,9 +3442,9 @@ static LLVMValueRef visit_image_atomic(struct
>> nir_to_llvm_context *ctx,
>>                 abort();
>>         }
>>
>> -       params[param_count++] = get_src(ctx, instr->src[2]);
>>         if (instr->intrinsic == nir_intrinsic_image_atomic_comp_swap)
>>                 params[param_count++] = get_src(ctx, instr->src[3]);
>> +       params[param_count++] = get_src(ctx, instr->src[2]);
>>
>>         if (glsl_get_sampler_dim(type) == GLSL_SAMPLER_DIM_BUF) {
>>                 params[param_count++] = get_sampler_desc(ctx,
>> instr->variables[0],  DESC_BUFFER);
>> --
>> 2.9.4
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170630/b6cb1d5b/attachment-0001.html>


More information about the mesa-dev mailing list