This patch is Reviewed-by: Ian Romanick <[email protected]>
On 03/23/2015 05:47 AM, [email protected] wrote: > From: Ville Syrjälä <[email protected]> > > The sub-pixel adjustment for points was killed off in > commit 60d762aa625095a8c1f9597d8530bb5a6fa61b4c > Author: Xiang, Haihao <[email protected]> > Date: Wed Jan 2 11:38:51 2008 +0800 > > i915: Needn't adjust pixel centers. fix #12944 > > so we can just as well use COPY_DWORDS(). > > Signed-off-by: Ville Syrjälä <[email protected]> > --- > src/mesa/drivers/dri/i915/intel_tris.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/src/mesa/drivers/dri/i915/intel_tris.c > b/src/mesa/drivers/dri/i915/intel_tris.c > index 2d91ca8..e46b142 100644 > --- a/src/mesa/drivers/dri/i915/intel_tris.c > +++ b/src/mesa/drivers/dri/i915/intel_tris.c > @@ -457,11 +457,7 @@ intel_draw_point(struct intel_context *intel, > intelVertexPtr v0) > GLuint *vb = intel_get_prim_space(intel, 1); > int j; > > - /* Adjust for sub pixel position -- still required for conform. */ > - *(float *) &vb[0] = v0->v.x; > - *(float *) &vb[1] = v0->v.y; > - for (j = 2; j < vertsize; j++) > - vb[j] = v0->ui[j]; > + COPY_DWORDS(j, vb, vertsize, v0); > } > > > _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
