On Friday, September 12, 2014 08:37:11 AM Ian Romanick wrote:
> On 09/10/2014 03:41 PM, Kenneth Graunke wrote:
> > We always uploaded them together, mostly out of laziness - both required
> > an additional vertex element.  However, gl_VertexID now also requires an
> > additional vertex buffer for storing gl_BaseVertex; for non-indirect
> > draws this also means uploading (a small amount of) data.  This is extra
> > overhead we don't need if the shader only uses gl_InstanceID.
> > 
> > In particular, our clear shaders currently use gl_InstanceID for doing
> > layered clears, but don't need gl_VertexID.
> > 
> > XXX: Needs testing on Broadwell before pushing.
> 
> Has this happened yet?

Yes, it seems fine.  "arb_transform_feedback2-draw-auto -auto instanced" 
appeared to fail originally, which was creepy, but subsequent runs show it 
passing, so I guess it must have been a fluke.

--Ken

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to