[Mesa-dev] [PATCH] nir/gcm: Add some missing break statements

Connor Abbott cwabbott0 at gmail.com
Sun Feb 22 14:38:20 PST 2015


Reviewed-by: Connor Abbott <cwabbott0 at gmail.com>

Whoops, I should've caught that...

On Sun, Feb 22, 2015 at 1:31 AM, Jason Ekstrand <jason at jlekstrand.net> wrote:
> ---
>  src/glsl/nir/nir_opt_gcm.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/glsl/nir/nir_opt_gcm.c b/src/glsl/nir/nir_opt_gcm.c
> index bf565b9..b4f5fd3 100644
> --- a/src/glsl/nir/nir_opt_gcm.c
> +++ b/src/glsl/nir/nir_opt_gcm.c
> @@ -121,9 +121,11 @@ gcm_pin_instructions_block(nir_block *block, void *void_state)
>           case nir_op_fddy_coarse:
>              /* These can only go in uniform control flow; pin them for now */
>              instr->pass_flags = GCM_INSTR_PINNED;
> +            break;
>
>           default:
>              instr->pass_flags = 0;
> +            break;
>           }
>           break;
>
> @@ -134,9 +136,11 @@ gcm_pin_instructions_block(nir_block *block, void *void_state)
>           case nir_texop_lod:
>              /* These two take implicit derivatives so they need to be pinned */
>              instr->pass_flags = GCM_INSTR_PINNED;
> +            break;
>
>           default:
>              instr->pass_flags = 0;
> +            break;
>           }
>           break;
>
> --
> 2.3.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list