Re: [Mesa-dev] [PATCH 2/6] nir/lower_io_to_temporaries: don't set compact on shadow vars

2017-07-06 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Thu, Jul 6, 2017 at 12:48 PM, Connor Abbott wrote: > From: Connor Abbott > > The compact flag doesn't make sense on local variables, since the > packing on them is up to the driver. This fixes nir_validate assertions > in some cases, particularly when lower_io_to

[Mesa-dev] [PATCH 2/6] nir/lower_io_to_temporaries: don't set compact on shadow vars

2017-07-06 Thread Connor Abbott
From: Connor Abbott The compact flag doesn't make sense on local variables, since the packing on them is up to the driver. This fixes nir_validate assertions in some cases, particularly when lower_io_to_temporaries is used on per-vertex inputs/outputs. --- src/compiler/nir/nir_lower_io_to_tempor