On 2018-02-09 — 12:06, Timothy Arceri wrote: > --- > src/gallium/drivers/r600/r600_pipe.c | 11 ++++++++--- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/drivers/r600/r600_pipe.c > b/src/gallium/drivers/r600/r600_pipe.c > index 6c021e568d..719ab349ef 100644 > --- a/src/gallium/drivers/r600/r600_pipe.c > +++ b/src/gallium/drivers/r600/r600_pipe.c > @@ -600,10 +600,15 @@ static int r600_get_shader_param(struct pipe_screen* > pscreen, > } else { > return PIPE_SHADER_IR_TGSI; > } > - case PIPE_SHADER_CAP_SUPPORTED_IRS: > + return PIPE_SHADER_IR_TGSI;
This line is dead code and can be safely removed, as both the if-block and the else-block return a value already; it should be part of patch 03 instead. With that addressed, this patch is Acked-by: Pierre Moreau <pierre.mor...@free.fr> > + case PIPE_SHADER_CAP_SUPPORTED_IRS: { > + int ir = 0; > + if (shader == PIPE_SHADER_COMPUTE) > + ir = 1 << PIPE_SHADER_IR_NATIVE; > if (rscreen->b.family >= CHIP_CEDAR) > - return (1 << PIPE_SHADER_IR_TGSI); > - return 0; > + ir |= 1 << PIPE_SHADER_IR_TGSI; > + return ir; > + } > case PIPE_SHADER_CAP_TGSI_FMA_SUPPORTED: > if (rscreen->b.family == CHIP_ARUBA || > rscreen->b.family == CHIP_CAYMAN || > -- > 2.14.3 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev