On Sat, Apr 14, 2018 at 3:39 PM, Erico Nunes <[email protected]> wrote: > On Sat, Apr 14, 2018 at 9:26 PM, Jason Ekstrand <[email protected]> wrote: >> Reviewed-by: Jason Ekstrand <[email protected]> >> >> What driver is hitting this path? The !supports_ints path isn't used to my >> knowledge so if some driver has started using it, they're liable to find >> more bugs than just this one. :-) > > I'm doing some work on the lima vertex shader compiler and I hit this. > > And yeah this is there since 2015 it seems, so I suppose no other > drivers are using this path, we'll see if there's more.
I think that it's probably impractical to use this path, and we should probably delete it. There are just too many optimizations, e.g. in nir_opt_algebraic and lowering passes that assume you have ints. I think a better plan would be to silently convert ints to floats in the lima driver, and maybe inhibit any optimizations that use bit twiddling tricks if real int support isn't indicated. > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
