projects / platform / upstream / mesa.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: c173541) | patch
i965/fs: Check for compilation failure and bail before optimizing.
authorKenneth Graunke <kenneth@whitecape.org>
Fri, 10 Jun 2011 23:00:03 +0000 (16:00 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Sun, 19 Jun 2011 00:53:24 +0000 (17:53 -0700)
commitad9481e12813d5f1dec95ce123927e132fa935fb
tree79c2a91fd6fb1f93c22cc36bd7943d83084dbd9dtree | snapshot
parentc173541d9769d41a85cc899bc49699a3587df4bfcommit | diff
i965/fs: Check for compilation failure and bail before optimizing.

Prior to this patch, it would attempt to optimize and allocate registers
for the program even if it failed to compile.  This seems wasteful.

More importantly, the "message length > 11" failure seems to choke the
instruction scheduler, making it somehow use an undefined value and
segmentation fault.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/drivers/dri/i965/brw_fs.cpp diff | blob | history
Domain: Graphics System / GL;