[Mesa-dev] [PATCH] Revert "st/nir: use NIR for asm programs"
eric at anholt.net
Wed May 30 18:54:52 UTC 2018
Timothy Arceri <tarceri at itsqueeze.com> writes:
> On 22/05/18 04:15, Eric Anholt wrote:
>> Timothy Arceri <tarceri at 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  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:
>> 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 <tarceri at itsqueeze.com>
Thanks! And thanks for applying while I was gone.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 832 bytes
Desc: not available
More information about the mesa-dev