[Mesa-dev] [PATCH 00/19] i965/fs: load_payload on Gen7+.

[Matt Turner]

On Tue, Jun 10, 2014 at 4:41 PM, Matt Turner <mattst88 at gmail.com> wrote:
> On Tue, May 27, 2014 at 6:47 PM, Matt Turner <mattst88 at gmail.com> wrote:
>> Here's a respin of my load_payload series from mid-April with some
>> feedback from Ken addressed and some bugs fixed.
>>
>>    git://people.freedesktop.org/~mattst88/mesa tex-sources
>
> I've force pushed an updated and rebased branch.
>
>> I'd really like to get some R-b tags this time around, since this
>> series is necessary for the SSA work I've been anxious to get back to
>> for a while.
>>
>> i965/fs: Add and use an fs_inst copy constructor.
>> i965/fs: Disable fs_inst assignment operator.
>> i965/fs: ralloc fs_inst's fs_reg sources.
>> i965/fs: Store the number of sources an fs_inst has.
>> i965/fs: Loop from 0 to inst->sources, not 0 to 3.
>> i965/fs: Clean up fs_inst constructors.
>> i965/fs: Add a function to resize fs_inst's sources
>> i965/fs: Add fs_inst constructor that takes a list of
>
> These were reviewed and committed.
>
>> i965/fs: Add SHADER_OPCODE_LOAD_PAYLOAD.
>
> This has a review.
>
>> i965/fs: Lower LOAD_PAYLOAD and clean up.
>> i965/fs: Use LOAD_PAYLOAD in emit_texture_gen7().
>> i965/fs: Apply cube map array fixup and restore the
>> i965/fs: Only consider real sources when comparing
>> i965/fs: Emit load_payload instead of multiple MOVs for
>> i965/fs: Support register coalescing on LOAD_PAYLOAD
>> i965/fs: Perform CSE on load_payload instructions if
>> i965/fs: Copy propagate from load_payload.
>
> These are missing reviews. I've made some small changes in response to
> some comments and mailed some fixups that will be squashed into their
> respective patches. I'd really like to get this stuff committed.
> Should I hope and wait for more review? Resend the whole set of
> unreviewed patches?

Didn't hear anything. Pushed.

Let me know if there are any problems.

>> i965/fs: Perform CSE on texture operations.
>> i965/fs: Optimize SEL with the same sources into a MOV.
>
> These two have reviews.