I haven't encountered such dereference issues, but lowering integers later is a good idea (as with bools which are now lowered later).

On 12/19/2018 01:22 PM, Eric Anholt wrote:
Jonathan Marek <jonat...@marek.ca> writes:

Note: the backend must take care that uniform index is now a float

This makes me think that lowering ints to float should be done near the
end of the compile (followed by maybe an algebraic and a dce).  As is, I
think nir_lower_io() is going to do bad things to dereferences of i/o
arrays.

That said, it looks like this will be fixing way more than it regresses,
so I would go along with it.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to