diff options
author | Ian Romanick <[email protected]> | 2011-07-25 15:58:07 -0700 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2011-08-02 08:23:15 -0700 |
commit | 322c3bf9dc4c6edbf5a8793475ce1307e1c0186b (patch) | |
tree | 98c8517be3699a6cff030184e40c1206e11e2008 /src/mesa/main/arbprogram.c | |
parent | 8aadd89d07d750aadd10989fa9c81f8a2fdd98e2 (diff) |
ir_to_mesa: Emit warnings instead of errors for IR that can't be lowered
Rely on the driver to do the right thing. This probably means falling
back to software. Page 88 of the OpenGL 2.1 spec specifically says:
"A shader should not fail to compile, and a program object should
not fail to link due to lack of instruction space or lack of
temporary variables. Implementations should ensure that all valid
shaders and program objects may be successfully compiled, linked
and executed."
There is no provision for saying "No" to a valid shader that is
difficult for the hardware to handle, so stop doing that.
On i915 this causes a large number of piglit tests to change from FAIL
to WARN. The warning is because the driver still emits messages to
stderr like "i915_program_error: Unsupported opcode: BGNLOOP".
It also fixes ES2 conformance CorrectFull_frag and CorrectParse1_frag
on i915 (and probably other hardware that can't handle loops).
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/mesa/main/arbprogram.c')
0 files changed, 0 insertions, 0 deletions