On Tuesday, September 5, 2017 8:19:57 PM PDT Dave Airlie wrote:
> From: Dave Airlie <airl...@redhat.com>
>
> This being declared bool means it won't get merged with the previous
> bitfields, this seems like an oversight rather than deliberate.
>
> Noticed when running pahole.
>
> Signed-off-by: Dave Airlie <airl...@redhat.com>
> ---
> src/compiler/nir/nir.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
> index 9313b7ac90..8330e6d7ce 100644
> --- a/src/compiler/nir/nir.h
> +++ b/src/compiler/nir/nir.h
> @@ -220,7 +220,7 @@ typedef struct nir_variable {
> * be tightly packed. In other words, consecutive array elements
> * should be stored one component apart, rather than one slot apart.
> */
> - bool compact:1;
> + unsigned compact:1;
>
> /**
> * Whether this is a fragment shader output implicitly initialized with
> This is fine, but I'd prefer to change the other booleans to "bool" in the long term.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
