Patch 1 fixes some compiler asserts I was running into: Maybe we can just do those optimizations anyway, but simply drop the dnz flag on the ADD as long as the instructions aren't marked as being prices
Patch 2 tries to fix our outstanding issue with bound samplers with nine.: I don't really know if this is the correct fix for it, but it makes sense to me reading the commit message of the gallium change. No piglit regressions and games are working again using gallium nine and nouveau. Maybe we should drop that entire num_samples handling alltogether? Karol Herbst (2): nv50/ir: don't optimize dnz muls to add nv50,nvc0: Fix gallium nine regression regarding sampler bindings .../nouveau/codegen/nv50_ir_peephole.cpp | 6 ++++-- src/gallium/drivers/nouveau/nv50/nv50_state.c | 13 ++----------- src/gallium/drivers/nouveau/nvc0/nvc0_state.c | 17 +++++------------ 3 files changed, 11 insertions(+), 25 deletions(-) -- 2.19.1 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev