[Mesa-dev] [PATCH 1/3] i965/nir: Add support for nir_intrinsic_load_front_face

Jason Ekstrand jason at jlekstrand.net
Sat Feb 14 13:14:55 PST 2015


On Sat, Feb 14, 2015 at 1:02 PM, Matt Turner <mattst88 at gmail.com> wrote:

> On Sat, Feb 14, 2015 at 1:00 PM, Connor Abbott <cwabbott0 at gmail.com>
> wrote:
> > I think we should be adding this to nir_emit_system_values_block()
> > like we do for the other system values. This should get us parity with
> > what we had before where we only called
> > emit_frontfacing_interpolation() once per shader.
>
> Calling it once per shader isn't really important -- it just emits a
> single instruction.
>

In fact, we should probably be careful with that in general as it lengthens
live-ranges.  Oh, for GCM in the backend IR.
--Jason

In fact, once we remove brw_fs_visitor.cpp I'd be in favor of removing
> emit_frontfacing_interpolation() itself.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150214/f1e8dec6/attachment-0001.html>


More information about the mesa-dev mailing list