On 01/06/2012 04:49 PM, Ian Romanick wrote:
Patches 01/14 through 03/14 was previously sent to the list.  It
tickled a pre-existing bug (fixed by patch recently posted to the list
by Eric) on some Intel chipsets.

Patches 04/14 through 07/14 make the linker do some sampler tracking
that was previously done by ir_to_mesa.  I suspect that the same work
is duplicated by st_glsl_to_tgsi, so some refactoring could probably
happen there.

The remaining patches gut the pre-compile paths and the use of
ir_to_mesa from the i965 driver.  There are a couple state bits that
were previously calculated by ir_to_mesa that now must be tracked by
the driver (see patches 12/14 and 14/14).  It is arguable that these
two patches should be squashed into 13/14.  I sent them separately to
make the changes more explicit.

I observe no piglit regressions on SNB with this series.  I don't have
convenient access to other hardware, so I haven't tested that yet.  I
plan to do that testing before pushing.

If it's helpful, Dylan has G33, Pineview, G965, and GM45 set up in the lab and running tests. You just drop in your git remote URL and branch name and it runs Piglit on all your platforms. (No Ironlake or Sandybridge yet, unfortunately...we need to buy more machines.)

Patches 1, 3, and 13 are
Acked-by: Kenneth Graunke <kenn...@whitecape.org>

Patches 2, 4-7, and 11 are
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

I am reluctant about 8-10 (replied separately); I also think 12 and 14 could be done better but if there isn't time, then oh well.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to