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

Matt Turner mattst88 at gmail.com
Tue Jun 10 16:41:12 PDT 2014


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?

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

These two have reviews.


More information about the mesa-dev mailing list