Re: [Mesa-dev] [PATCH 1/2] nir: Add a safety check that we don't remove dead I/O vars after lowering.

2017-10-17 Thread Eric Anholt
Timothy Arceri writes: > On 18/10/17 07:52, Eric Anholt wrote: >> The pass only looks at var load/store intrinsics, not input load/store >> intrinsics, so assert that we don't see the other type. >> --- >> >> I tripped over this limitation when trying to use the NIR

Re: [Mesa-dev] [PATCH 1/2] nir: Add a safety check that we don't remove dead I/O vars after lowering.

2017-10-17 Thread Timothy Arceri
On 18/10/17 07:52, Eric Anholt wrote: The pass only looks at var load/store intrinsics, not input load/store intrinsics, so assert that we don't see the other type. --- I tripped over this limitation when trying to use the NIR linking helpers in vc4.

[Mesa-dev] [PATCH 1/2] nir: Add a safety check that we don't remove dead I/O vars after lowering.

2017-10-17 Thread Eric Anholt
The pass only looks at var load/store intrinsics, not input load/store intrinsics, so assert that we don't see the other type. --- I tripped over this limitation when trying to use the NIR linking helpers in vc4. src/compiler/nir/nir_remove_dead_variables.c | 20 1 file