From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Anholt Subject: Re: [Mesa-dev] [PATCH 5/9] intel gen4-5: Compute the interpolation status for every variable in one place. Date: Thu, 26 Jul 2012 10:22:26 -0700 Message-ID: <87wr1qphd9.fsf@eliezer.anholt.net> References: <1342728024-15055-1-git-send-email-galibert@pobox.com> <1342728024-15055-6-git-send-email-galibert@pobox.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0344062921==" Return-path: In-Reply-To: <1342728024-15055-6-git-send-email-galibert@pobox.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: intel-gfx@lists.freedesktop.org, mesa-dev@lists.freedesktop.org Cc: Olivier Galibert List-Id: intel-gfx@lists.freedesktop.org --===============0344062921== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature" --=-=-= Content-Transfer-Encoding: quoted-printable Olivier Galibert writes: > The program keys are updated accordingly, but the values are not used > yet. > > Signed-off-by: Olivier Galibert > --- > src/mesa/drivers/dri/i965/brw_clip.c | 90 +++++++++++++++++++++++++= +++++- > src/mesa/drivers/dri/i965/brw_clip.h | 1 + > src/mesa/drivers/dri/i965/brw_context.h | 11 ++++ > src/mesa/drivers/dri/i965/brw_sf.c | 5 +- > src/mesa/drivers/dri/i965/brw_sf.h | 1 + > src/mesa/drivers/dri/i965/brw_wm.c | 2 + > src/mesa/drivers/dri/i965/brw_wm.h | 1 + > 7 files changed, 109 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_clip.c b/src/mesa/drivers/dri/= i965/brw_clip.c > index d411208..b4a2e0a 100644 > --- a/src/mesa/drivers/dri/i965/brw_clip.c > +++ b/src/mesa/drivers/dri/i965/brw_clip.c > @@ -47,6 +47,86 @@ > #define FRONT_UNFILLED_BIT 0x1 > #define BACK_UNFILLED_BIT 0x2 >=20=20 > +/** > + * Lookup the interpolation mode information for every element in the > + * vue. > + */ > +static void > +brw_lookup_interpolation(struct brw_context *brw) > +{ > + /* pprog means "previous program", i.e. the last program before the > + * fragment shader. It can only be the vertex shader for now, but > + * it may be a geometry shader in the future. > + */ > + const struct gl_program *pprog =3D &brw->vertex_program->Base; > + const struct gl_fragment_program *fprog =3D brw->fragment_program; > + struct brw_vue_map *vue_map =3D &brw->vs.prog_data->vue_map; > + > + /* Default everything to INTERP_QUALIFIER_NONE */ > + memset(brw->interpolation_mode, INTERP_QUALIFIER_NONE, BRW_VERT_RESUL= T_MAX); I don't like seeing this data that should be referenced out of the program cache key being communicated through brw->. --=-=-= Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iEYEARECAAYFAlARfNIACgkQHUdvYGzw6vfJpACeJJSkt5apNPDViMS30K37PSQv 63cAoJfOdfEYQlzp+t0M6qa0wTy0X9Ie =HRAa -----END PGP SIGNATURE----- --=-=-=-- --===============0344062921== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============0344062921==--