glsl2: Remove the shader_in/shader_out tracking separate from var->mode.
authorEric Anholt <eric@anholt.net>
Thu, 5 Aug 2010 03:33:57 +0000 (20:33 -0700)
committerEric Anholt <eric@anholt.net>
Thu, 5 Aug 2010 03:52:33 +0000 (20:52 -0700)
commit046bef235744e891e4a48076e1a3ff9a61a63092
tree1ddd953a9e7ec14e1726d16011985910e372742a
parenta08f27940ac72538ce0b264917207111d629f097
glsl2: Remove the shader_in/shader_out tracking separate from var->mode.

I introduced this for ir_dead_code to distinguish function parameter
outvals from varying outputs.  Only, since ast_to_hir's
current_function is unset when setting up function parameters (they're
needed for making the function signature in the first place), all
function parameter outvals were marked as shader outputs anyway.  This
meant that an inlined function's cloned outval was marked as a shader
output and couldn't be dead-code eliminated.  Instead, since
ir_dead_code doesn't even look at function parameters, just use
var->mode.

The longest Mesa IR coming out of ir_to_mesa for Yo Frankie drops from
725 instructions to 636.
src/glsl/ast_to_hir.cpp
src/glsl/ir.cpp
src/glsl/ir.h
src/glsl/ir_clone.cpp
src/glsl/ir_dead_code.cpp
src/glsl/ir_variable.cpp
src/glsl/linker.cpp