[Mesa-dev] V3 Shared shader info

Jason Ekstrand jason at jlekstrand.net
Fri Oct 21 04:28:37 UTC 2016


1-27/42 and 29-42/42 are

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

28/42 will take more brain power than I can exert tonight.  I'll look at it
tomorrow.

Thanks for working on this.  I know how much pain refactoring can be.

On Wed, Oct 19, 2016 at 4:09 PM, Timothy Arceri <
timothy.arceri at collabora.com> wrote:

> V3:
> - create a temporary copy_shader_info() helper and split up the changes
>   in this series for easier review as suggested by Jason/Ken.
> - a couple of small changes as suggested by Ian and Marek (more details in
>   patches).
>
> Having the shader info struct in nir_shader makes things difficult for
> the shader cache, and means we are copying values in the glsl->nir pass
> unnecessarily. This series creates a shared shader info and makes a bunch
> of clean-ups based on this change.
>
> The st changes have been tested against llvmpipe.
>
> There is still more work to be done to remove some of the now duplicate
> fields in gl_program but I'll send that in a follow up series.
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161020/f260e86a/attachment.html>


More information about the mesa-dev mailing list