On 02/12/2015 12:13 PM, Ian Romanick wrote: > Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
And please add Cc: "10.4 10.5" <mesa-sta...@lists.freedesktop.org> > 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 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
