On 22/05/18 04:15, Eric Anholt wrote:
Timothy Arceri <tarc...@itsqueeze.com> writes:

On 18/05/18 00:53, Eric Anholt wrote:
This reverts commit 5c33e8c7729edd5e16020ebb8703be96523e04f2.  It broke
fixed function vertex programs on vc4 and v3d, and apparently caused
trouble for radeonsi's NIR paths as well.
Has someone reported trouble with radeonsi NIR? I'm not aware of any
issues. Dave's patch [1] was for fixing iris, I had no way to test so
didn't try send it out myself after you confirmed it fixed your issue.
OK, I had that mistaken.

Still, an unexplained workaround on a branch is not a great response to
this.  And even with that workaround plus your change, the following
tests are still broken on V3D:

gl-1.0-rendermode-fallback
opengl-1.1/gl_select-*
fp-arb-fragment-coord-conventions-integer.shader_test
vp-max-array
ati_fragment_shader-render-constants and 6 others.

I'd like to land the revert until there are clean piglit results.
Sure. As I said I have no idea why the workaround helps, seems radeonsi 
does something different to the other drivers. For now feel free to push 
the revert.
Acked-by: Timothy Arceri <tarc...@itsqueeze.com>

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to