Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> On 02/12/2015 11:46 AM, Carl Worth wrote: > Commit f82f2fb3dc770902f1657ab1c22e6004faa3afab added use of the Mesa > IR optimizer for both ARB_fragment_program and ARB_vertex_program, but > only justified the vertex-program portions with measured performance > improvements. > > Meanwhile, the optimizer was seen to generate hundreds of unused > immediates without discarding them, causing failures. > > Discard the use of the optimizer for now to fix the regression. (In > the future, we anticpate things moving from Mesa IR to NIR for better > optimization anyway.) > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82477 > --- > src/mesa/program/arbprogparse.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/src/mesa/program/arbprogparse.c b/src/mesa/program/arbprogparse.c > index 7dec399..53a6f37 100644 > --- a/src/mesa/program/arbprogparse.c > +++ b/src/mesa/program/arbprogparse.c > @@ -85,9 +85,6 @@ _mesa_parse_arb_fragment_program(struct gl_context* ctx, > GLenum target, > return; > } > > - if ((ctx->_Shader->Flags & GLSL_NO_OPT) == 0) > - _mesa_optimize_program(ctx, &prog); > - > free(program->Base.String); > > /* Copy the relevant contents of the arb_program struct into the >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
