[Mesa-dev] [PATCH 24/40] i965/meta: Expose fast clear rectangle calculation

Pohjolainen, Topi topi.pohjolainen at intel.com
Thu Apr 21 04:14:58 UTC 2016


On Wed, Apr 20, 2016 at 09:06:57PM -0700, Kenneth Graunke wrote:
> On Saturday, April 16, 2016 4:42:52 PM PDT Topi Pohjolainen wrote:
> > Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
> > ---
> >  src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 44 +++++++++++++
> +-----------
> >  src/mesa/drivers/dri/i965/brw_meta_util.h       |  8 +++++
> >  2 files changed, 33 insertions(+), 19 deletions(-)
> > 
> > diff --git a/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c b/src/mesa/
> drivers/dri/i965/brw_meta_fast_clear.c
> > index 1fb5dc8..885c09c 100644
> > --- a/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c
> > +++ b/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c
> > @@ -58,6 +58,7 @@
> >  #include "intel_batchbuffer.h"
> >  
> >  #include "brw_blorp.h"
> > +#include "brw_meta_util.h"
> >  
> >  struct brw_fast_clear_state {
> >     struct gl_buffer_object *buf_obj;
> > @@ -171,7 +172,7 @@ brw_meta_fast_clear_free(struct brw_context *brw)
> >  }
> >  
> >  struct rect {
> > -   int x0, y0, x1, y1;
> > +   unsigned x0, y0, x1, y1;
> 
> Any particular reason for the type change?  The things being assigned to
> it are GLints...

In blorp they are of the type uint32_t, so I would need to change one or the
other. As these values can never be negative I thought keeping to blorp was
more fitting.


More information about the mesa-dev mailing list