draw: don't use clipvertex output if user plane clipping is disabled
authorRoland Scheidegger <sroland@vmware.com>
Sat, 9 Aug 2014 01:51:23 +0000 (03:51 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Sat, 9 Aug 2014 01:52:58 +0000 (03:52 +0200)
The non-llvm path made sure that both clip and pre_clip_pos point to the data
output by position, not clipvertex, if user based clipping is disabled.
However, the llvm path did not, which apparently led to failures if
gl_ClipVertex was written but user plane clipping not enabled (bug 80183).
Why I have no idea really, but just make it match the non-llvm behavior...

Reviewed-by: Brian Paul <brianp@vmware.com>
src/gallium/auxiliary/draw/draw_llvm.c

index d29adfbea3ec18a01a1a24d84ec008465b9796da..967eb7e017327e58025e0ad47007cc6c1a5989c1 100644 (file)
@@ -1732,8 +1732,8 @@ draw_llvm_generate(struct draw_llvm *llvm, struct draw_llvm_variant *variant,
 
       if (pos != -1 && cv != -1) {
          /* store original positions in clip before further manipulation */
-         store_clip(gallivm, vs_type, io, outputs, 0, cv);
-         store_clip(gallivm, vs_type, io, outputs, 1, pos);
+         store_clip(gallivm, vs_type, io, outputs, FALSE, key->clip_user ? cv : pos);
+         store_clip(gallivm, vs_type, io, outputs, TRUE, pos);
 
          /* do cliptest */
          if (enable_cliptest) {