[Mesa-dev] [PATCH] ac/nir: add break statements in needs_view_index_sgpr()

[Bas Nieuwenhuizen]

Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

On Thu, Jan 25, 2018 at 10:49 AM, Samuel Pitoiset <samuel.pitoiset at gmail.com
> wrote:

> Previous code is correct but as the first case statement uses
> a break, keep it consistent.
>
> CID: 1428579
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  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
> 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/20180125/cdddf88f/attachment.html>