[Mesa-dev] [PATCH 2/2] ir_to_mesa: Let check_resources halt compilation
kenneth at whitecape.org
Thu Oct 27 22:32:59 PDT 2011
On 10/27/2011 02:59 PM, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
> Previously check_resources could fail, but we'd still try to optimize
> the shader, do device-specific code generation, etc. In some cases,
> this could explode (especially in the device-specific code
> generation). I haven't found that I could trigger this with the
> current code.
Really? Try running oglconform...or the attached shader_runner test.
> When too many samplers were used with the new uniform
> handling code, I observed several crashes deep down in the driver.
Yeah. Basically anything that fails to link crashes in
brw_fs_precompile with current code (master).
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41609
> Cc: Eric Anholt <eric at anholt.net>
For patch 1, you get a
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
For patch 2, you get a whopping
Reviewed-and-tested-by: Kenneth Graunke <kenneth at whitecape.org>
(it fixes the crash in the attached test)
Thanks for fixing this, Ian.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the mesa-dev