>> + >> + assert(var); >> + location = var->data.location; >> + calc_deref_offsets(ir, ir, array_size, base, index, reladdr, >> &location); >> + >> + if (location != 0xffffffff) { > > This could just be an assert right? It shouldn't ever be -1 for a > sampler unless something went wrong in the linker.
atomics trigger this. so I pulled it into this patch. > > As I said last time I don't really understand all this tgsi specifics > (which is around half the patch) but this time round the offset > calculation looks correct so for what its worth this patch, 3 and 5 are > > Reviewed-by: Timothy Arcer <timothy.arc...@collabora.com> Thanks, Dave. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev