gallivm: some minor cube map cleanup
authorRoland Scheidegger <sroland@vmware.com>
Thu, 4 Apr 2013 21:20:49 +0000 (23:20 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Thu, 4 Apr 2013 21:22:10 +0000 (23:22 +0200)
commit9eef86bb55108bfe2537d90e7efb41a30bf8547a
treeb10bbcd549f21b08a7dd71a0cb0926bf91f9896c
parentb6e9b54d066d29d250c4502e0005b317589dd8b3
gallivm: some minor cube map cleanup

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.
And also change the chosen major axis in case the coord values are the
same. OpenGL doesn't care one bit which one is chosen in this case but
it looks like dx10 would require z chosen over y, and y chosen over x
(previously did x chosen over y, y chosen over z). Since it's all the
same effort just honor dx10's wishes. (Though actually, for some prefered
orderings, we could save one (or two with derivatives) selects since the
tnewx and tnewz (and the corresponding dmax values) are the same.)

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
src/gallium/auxiliary/gallivm/lp_bld_sample.c