nir/lower_variables: Follow the Cytron paper more closely
authorJason Ekstrand <jason.ekstrand@intel.com>
Thu, 18 Dec 2014 22:42:01 +0000 (14:42 -0800)
committerJason Ekstrand <jason.ekstrand@intel.com>
Thu, 15 Jan 2015 15:20:23 +0000 (07:20 -0800)
commitad9d0a9ea6d40db776e6680a44d614cd447dc17c
treee2e8a362ad6d84a4b1a31f6bfcbfdf1156da3ee9
parentb1d114a48c09306d18160e5b492c5f9db0bf6db2
nir/lower_variables: Follow the Cytron paper more closely

Previously, our variable renaming algorithm, while similar to the one in
the Cytron paper, was not the same.  While I'm pretty sure it was correct,
it will be easier for readers of the code in the variable renaming pass if
it follows more closely.  This commit removes the automatic stack popping
we were doing and replaces it with explicit popping like Cytron does.

Reviewed-by: Connor Abbott <cwabbott0@gmail.com>
src/glsl/nir/nir_lower_variables.c