diff options
| author | Carl Worth <cworth@cworth.org> | 2015-02-12 10:14:24 -0800 |
|---|---|---|
| committer | Carl Worth <cworth@cworth.org> | 2015-02-12 13:33:12 -0800 |
| commit | 55a57834bf912916a5e07486b32f1778b35657f6 (patch) | |
| tree | 18dcc91909f6cfbe12af5af11d56a0b2f42daa73 /src/mesa/program/arbprogparse.c | |
| parent | 1ba9f9e62c1e83a2a9a48435cce5128be3d3ea66 (diff) | |
| download | external_mesa3d-55a57834bf912916a5e07486b32f1778b35657f6.zip external_mesa3d-55a57834bf912916a5e07486b32f1778b35657f6.tar.gz external_mesa3d-55a57834bf912916a5e07486b32f1778b35657f6.tar.bz2 | |
Revert use of Mesa IR optimizer for ARB_fragment_programs
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
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
CC: "10.3 10.4 10.5" <mesa-stable@lists.freedesktop.org>
Diffstat (limited to 'src/mesa/program/arbprogparse.c')
| -rw-r--r-- | src/mesa/program/arbprogparse.c | 3 |
1 files changed, 0 insertions, 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 |