Reviewed-by: Ian Romanick <[email protected]>

On 03/12/2018 11:21 AM, Eric Anholt wrote:
> v2: Use assume() at the srcs[] definition instead.
> 
> Cc: Jason Ekstrand <[email protected]>
> Cc: Ian Romanick <[email protected]>
> Cc: Eric Engestrom <[email protected]>
> ---
>  src/compiler/spirv/spirv_to_nir.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/compiler/spirv/spirv_to_nir.c 
> b/src/compiler/spirv/spirv_to_nir.c
> index 6a358c597316..3de45c47371e 100644
> --- a/src/compiler/spirv/spirv_to_nir.c
> +++ b/src/compiler/spirv/spirv_to_nir.c
> @@ -2925,6 +2925,7 @@ vtn_handle_composite(struct vtn_builder *b, SpvOp 
> opcode,
>  
>     case SpvOpCompositeConstruct: {
>        unsigned elems = count - 3;
> +      assume(elems >= 1);
>        if (glsl_type_is_vector_or_scalar(type)) {
>           nir_ssa_def *srcs[4];
>           for (unsigned i = 0; i < elems; i++)
> 

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to