On Wednesday, June 18, 2014 01:45:52 PM Ian Romanick wrote: > On 06/18/2014 02:51 AM, Iago Toral Quiroga wrote: > > --- > > src/mesa/drivers/dri/i965/brw_context.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c > > index cfb0be0..6ba58d3 100644 > > --- a/src/mesa/drivers/dri/i965/brw_context.c > > +++ b/src/mesa/drivers/dri/i965/brw_context.c > > @@ -507,6 +507,10 @@ brw_initialize_context_constants(struct brw_context *brw) > > ctx->Const.ViewportBounds.Min = -(float)ctx- >Const.MaxViewportWidth; > > ctx->Const.ViewportBounds.Max = ctx->Const.MaxViewportWidth; > > } > > + > > + /* ARB_gpu_shader5 */ > > + if (brw->gen >= 7) > > + ctx->Const.MaxVertexStreams = MAX_VERTEX_STREAMS; > > I'd rather have this be MIN2(4, MAX_VERTEX_STREAMS). That way > increasing MAX_VERTEX_STREAMS to support some new hardware won't break GEN7.
That seems like a good idea. --Ken
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
