[Mesa-dev] [PATCH 08/17] tgsi/ureg: add shared variables support for compute shaders

[Ilia Mirkin]

On Tue, Jan 26, 2016 at 8:57 AM, Marek Olšák <maraeo at gmail.com> wrote:
> On Tue, Jan 26, 2016 at 2:25 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> I'd be fine with a new TGSI_FILE_MEMORY which provided options for
>> shared, global, and local(/private?) memory. I believe the old
>> TGSI_FILE_RESOURCE had support for these in a hacky way, this would be
>> the clean way of doing it.
>
> I think they mean:
> global = global shared memory
> local = shared within a thread group (GL "shared memory")
> private = ???

memory that is local to a thread.

>
> ureg_DECL_local_temporary seems like a good match. I'd prefer to have
> a separate file though.
>
> Shared memory is the same as TEMPs, except that they are TEMPs shared
> within a thread group.

It's much more of a memory area than a TEMP area though. TEMP's imply
16-byte wide stride for indirect indexing, etc -- not easy to work
with.

A file with ways to indicate global/shared/local seems better to me. I
thought we could just use BUFFER's for that, but if you prefer
yet-another file, that's fine too.

  -ilia