Re: [Mesa-dev] [PATCH] gallivm: some minor cube map cleanup

2013-04-04 Thread Jose Fonseca
Looks good Roland. - Original Message - > From: Roland Scheidegger > > The ar_ge_as_at variable was just very very confusing since the condition > was actually the other way around (as_at_ge_ar). So change the condition > (and the selects depending on it) to match the variable name. > Al

[Mesa-dev] [PATCH] gallivm: some minor cube map cleanup

2013-04-04 Thread sroland
From: Roland Scheidegger The ar_ge_as_at variable was just very very confusing since the condition was actually the other way around (as_at_ge_ar). So change the condition (and the selects depending on it) to match the variable name. Also, while here, change the chosen major axis in case the coor