Reviewed-by: Bas Nieuwenhuizen <[email protected]> On Thu, Jan 25, 2018 at 10:49 AM, Samuel Pitoiset <[email protected] > wrote:
> Previous code is correct but as the first case statement uses > a break, keep it consistent. > > CID: 1428579 > Signed-off-by: Samuel Pitoiset <[email protected]> > --- > src/amd/common/ac_nir_to_llvm.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_ > llvm.c > index 8ae8650a7b..7b56edf8e7 100644 > --- a/src/amd/common/ac_nir_to_llvm.c > +++ b/src/amd/common/ac_nir_to_llvm.c > @@ -555,10 +555,12 @@ static bool needs_view_index_sgpr(struct > nir_to_llvm_context *ctx, > case MESA_SHADER_TESS_EVAL: > if (ctx->shader_info->info.needs_multiview_view_index || > (!ctx->options->key.tes.as_es && ctx->options->key.has_ > multiview_view_index)) > return true; > + break; > case MESA_SHADER_GEOMETRY: > case MESA_SHADER_TESS_CTRL: > if (ctx->shader_info->info.needs_multiview_view_index) > return true; > + break; > default: > break; > } > -- > 2.16.1 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
