[Mesa-dev] [PATCH v3 00/10] Initial gl_spirv support in Mesa and i965

[Eduardo Lima Mitev]

Any chance to wrap up this review?

Thanks!

Eduardo


On 12/13/2017 08:32 PM, Eduardo Lima Mitev wrote:
> Hi,
>
> This is the 3rd version of the series adding initial support for ARB_gl_spirv.
>
> Previous versions of this series included also support for ARB_spirv_extensions, but we have decided to split the two to ease review. So I will be sending a second series with only the patches for spirv_extensions.
>
> Notice also that some patches from version 2 were merged in master. These were already reviewed favorably and were fairly independent from the rest of the series.
>
> There are still some patches in this new series with a Reviewed-by tag that we didn't merge yet because we consider they should go in with the rest of the series. The patches missing review are 01, 02, 03, 04 and 07.
>
> As usual, a git tree containing this series can be found at <https://github.com/Igalia/mesa/tree/arb_gl_spirv-series1-v3> and the larger, work-in-progress series at <https://github.com/Igalia/mesa/tree/wip/igalia/arb_gl_spirv>.
>
> Thanks for reviewing!
>
> cheers,
> Eduardo
>
> Alejandro Piñeiro (2):
>   i965: initialize SPIR-V capabilities
>   nir/spirv: add gl_spirv_validation method
>
> Eduardo Lima Mitev (6):
>   mesa: Add a reference to gl_shader_spirv_data to gl_linked_shader
>   mesa/glspirv: Add _mesa_spirv_link_shaders() function
>   mesa/program: Link SPIR-V shaders using the SPIR-V code-path
>   mesa/glspirv: Add a _mesa_spirv_to_nir() function
>   i965: Call spirv_to_nir() instead of glsl_to_nir() for SPIR-V shaders
>   i965: Don't call process_glsl_ir() for SPIR-V shaders
>
> Nicolai Hähnle (2):
>   mesa: add gl_constants::SpirVCapabilities
>   mesa: Implement glSpecializeShaderARB
>
>  src/compiler/spirv/nir_spirv.h          |   5 +
>  src/compiler/spirv/spirv_to_nir.c       | 191 +++++++++++++++++++++++---
>  src/mesa/drivers/dri/i965/brw_context.c |  20 +++
>  src/mesa/drivers/dri/i965/brw_link.cpp  |   3 +-
>  src/mesa/drivers/dri/i965/brw_program.c |  10 +-
>  src/mesa/main/glspirv.c                 | 236 +++++++++++++++++++++++++++++++-
>  src/mesa/main/glspirv.h                 |  11 ++
>  src/mesa/main/mtypes.h                  |  11 ++
>  src/mesa/main/shaderobj.c               |   1 +
>  src/mesa/program/ir_to_mesa.cpp         |   6 +-
>  10 files changed, 472 insertions(+), 22 deletions(-)
>