On Tue, 7 Jun 2011 11:47:13 -0700, Eric Anholt <e...@anholt.net> wrote: > This is already pointing at 0 or Height - 1 and with an appropriate > pitch, so no need to recompute those values per customization of the > spans code. Cuts 3 out of 21kb of the compiled size. > --- > src/mesa/drivers/dri/intel/intel_span.c | 11 ++++------- > 1 files changed, 4 insertions(+), 7 deletions(-) > > diff --git a/src/mesa/drivers/dri/intel/intel_span.c > b/src/mesa/drivers/dri/intel/intel_span.c > index 5290342..8978129 100644 > --- a/src/mesa/drivers/dri/intel/intel_span.c > +++ b/src/mesa/drivers/dri/intel/intel_span.c > @@ -50,21 +50,18 @@ intel_set_span_functions(struct intel_context *intel, > > #define LOCAL_VARS \ > struct intel_renderbuffer *irb = intel_renderbuffer(rb); \ > - const GLint yScale = rb->Name ? 1 : -1; \ > - const GLint yBias = rb->Name ? 0 : rb->Height - 1; > \ > int minx = 0, miny = 0; \ > int maxx = rb->Width; \ > int maxy = rb->Height; \ > - int pitch = irb->region->pitch * irb->region->cpp; > \ > - void *buf = irb->region->buffer->virtual; \ > + int pitch = rb->RowStride * irb->region->cpp; \ > + void *buf = rb->Data; \ > GLuint p; \ > - (void) p; \ > - (void)buf; (void)pitch; /* unused for non-gttmap. */ > \ > + (void) p;
Since you immediately void out p, why even declare it? > > #define HW_CLIPLOOP() > #define HW_ENDCLIPLOOP() > > -#define Y_FLIP(_y) ((_y) * yScale + yBias) > +#define Y_FLIP(_y) (_y) > > #define HW_LOCK() I also wanted to clean this up when defining the S8 span functions, but was afraidof breaking things. You, sir, are not afraid of breaking things. With the declaration of p killed, this is Reviewed-by: Chad Versace <c...@chad-versace.us> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev