[Mesa-dev] [PATCH 1/4] nir/split_var_copies: handle IMAGE and SAMPLER for bindless vars
Timothy Arceri
tarceri at itsqueeze.com
Wed Apr 4 10:33:15 UTC 2018
On 04/04/18 20:20, Karol Herbst wrote:
> On Wed, Apr 4, 2018 at 2:23 AM, Jason Ekstrand <jason at jlekstrand.net> wrote:
>> I have a very strong feeling that this isn't the only place where
>> reading/writing IMAGE and SAMPLER variables is going to cause NIR heartburn.
>> For example, we have special cases in nir_validate for SUBROUTINE variables
>> and we probably need IMAGE and SAMPLER support everywhere we have SUBROUTINE
>> plus some (since you can write to them now as well).
>>
>
> yeah. I was just making piglit happy here. I guess I will try to run
> it with some games using bindless_textures and fix all the crashes I
> encounter there at least. More piglit tests might be useful as well.
> Sadly I don't see any bindless_textures tests in the CTS :(
I don't think games really use the spec to its full extent either. I had
Dawn of War 3 working without issue with just the uniform support
implemented on radeonsi.
>
>>
>> On Tue, Apr 3, 2018 at 6:21 AM, Karol Herbst <kherbst at redhat.com> wrote:
>>>
>>> Signed-off-by: Karol Herbst <kherbst at redhat.com>
>>> ---
>>> src/compiler/nir/nir_split_var_copies.c | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/src/compiler/nir/nir_split_var_copies.c
>>> b/src/compiler/nir/nir_split_var_copies.c
>>> index bc3ceedbdb8..231a89add4d 100644
>>> --- a/src/compiler/nir/nir_split_var_copies.c
>>> +++ b/src/compiler/nir/nir_split_var_copies.c
>>> @@ -241,6 +241,10 @@ split_var_copies_block(nir_block *block, struct
>>> split_var_copies_state *state)
>>> ralloc_steal(state->dead_ctx, instr);
>>> }
>>> break;
>>> + /* for bindless those are uint64 */
>>> + case GLSL_TYPE_IMAGE:
>>> + case GLSL_TYPE_SAMPLER:
>>> + assert(src_head->var->data.bindless);
>>> case GLSL_TYPE_INT:
>>> case GLSL_TYPE_UINT:
>>> case GLSL_TYPE_INT16:
>>> --
>>> 2.14.3
>>>
>>> _______________________________________________
>>> mesa-dev mailing list
>>> mesa-dev at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>
>>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
More information about the mesa-dev
mailing list