FWIW, r600g passes the test and this commit has no effect on the driver. Marek
On Mon, Jul 2, 2012 at 1:24 PM, Jose Fonseca <jfons...@vmware.com> wrote: > This would affect not only llvmpipe but all gallium drivers, even those not > using draw module. > > Can other gallium driver maintainers acknowledge this patch also fixes their > test with their drivers? > > Jose > > ----- Original Message ----- >> That old bug was hidden but the clipper always interpolating in 3d >> space no matter what it should have been doing. Now that the >> interpolation has been fixed, the bug shows up. >> >> Fixes bugzilla 51364. >> >> Signed-off-by: Olivier Galibert <galib...@pobox.com> >> >> diff --git a/src/mesa/state_tracker/st_program.c >> b/src/mesa/state_tracker/st_program.c >> index e6664fb..9f98298 100644 >> --- a/src/mesa/state_tracker/st_program.c >> +++ b/src/mesa/state_tracker/st_program.c >> @@ -569,12 +569,12 @@ st_translate_fragment_program(struct st_context >> *st, >> case FRAG_ATTRIB_CLIP_DIST0: >> input_semantic_name[slot] = TGSI_SEMANTIC_CLIPDIST; >> input_semantic_index[slot] = 0; >> - interpMode[slot] = TGSI_INTERPOLATE_LINEAR; >> + interpMode[slot] = TGSI_INTERPOLATE_PERSPECTIVE; >> break; >> case FRAG_ATTRIB_CLIP_DIST1: >> input_semantic_name[slot] = TGSI_SEMANTIC_CLIPDIST; >> input_semantic_index[slot] = 1; >> - interpMode[slot] = TGSI_INTERPOLATE_LINEAR; >> + interpMode[slot] = TGSI_INTERPOLATE_PERSPECTIVE; >> break; >> /* In most cases, there is nothing special about these >> * inputs, so adopt a convention to use the generic >> _______________________________________________ >> mesa-dev mailing list >> mesa-dev@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/mesa-dev >> > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
