[Mesa-dev] [PATCH 0/5] nir, spirv: Fix bugs uncovered by spirv-opt
Iago Toral
itoral at igalia.com
Tue Oct 2 10:56:28 UTC 2018
On Tue, 2018-10-02 at 12:54 +0200, Iago Toral wrote:
> Letf a couple of minor comments, but otherwise the series is:
>
> Reviewed-by: Iago Toral Quiorga <itoral at igalia.com>
s/Quiorga/Quiroga
> On Sat, 2018-09-22 at 16:39 -0500, Jason Ekstrand wrote:
> > This little series fixes three bugs encountered while running the
> > Vulkan
> > CTS with SPIR-V optimizations enabled. The optimizations
> > shouldn't,
> > in
> > theory, change the behavior of the shader but it does change some
> > of
> > the
> > SPIR-V patterns and triggered a number of compiler bugs. I'd like
> > to
> > CC
> > the whole thing to stable but the last three patches are probably
> > best only
> > hitting the latest release because they're fairly major surgery.
> >
> > Jason Ekstrand (5):
> > nir/cf: Remove phi sources if needed in nir_handle_add_jump
> > nir/from_ssa: Don't rewrite derefs destinations to registers
> > spirv: Move function call handling to vtn_cfg
> > spirv: Pass SSA values through functions
> > spirv: Make images, samplers, and sampled images normal SSA
> > values
> >
> > src/compiler/nir/nir_control_flow.c | 36 ++--
> > src/compiler/nir/nir_from_ssa.c | 6 +
> > src/compiler/spirv/spirv_to_nir.c | 123 +++---------
> > src/compiler/spirv/vtn_cfg.c | 279 ++++++++++++++++++++++--
> > ----
> > src/compiler/spirv/vtn_private.h | 9 +-
> > src/compiler/spirv/vtn_variables.c | 51 +++--
> > 6 files changed, 297 insertions(+), 207 deletions(-)
More information about the mesa-dev
mailing list