nv50/ir,nvc0: make ClipDistance and ClipVertex work

3 files changed, 35 insertions, 10 deletions

diff --git a/src/gallium/drivers/nv50/codegen/nv50_ir_driver.h b/src/gallium/drivers/nv50/codegen/nv50_ir_driver.h
index a6a5e93f58c..4baef537370 100644
--- a/src/gallium/drivers/nv50/codegen/nv50_ir_driver.h
+++ b/src/gallium/drivers/nv50/codegen/nv50_ir_driver.h
@@ -152,8 +152,9 @@ struct nv50_ir_prog_info
 
    struct {
       uint8_t clipDistance;      /* index of first clip distance output */
-      uint8_t clipDistanceCount;
+      uint8_t clipDistanceMask;  /* mask of clip distances defined */
       uint8_t cullDistanceMask;  /* clip distance mode (1 bit per output) */
+      int8_t genUserClip;        /* request user clip planes for ClipVertex */
       uint8_t pointSize;         /* output index for PointSize */
       uint8_t edgeFlagIn;
       uint8_t edgeFlagOut;
diff --git a/src/gallium/drivers/nv50/codegen/nv50_ir_from_sm4.cpp b/src/gallium/drivers/nv50/codegen/nv50_ir_from_sm4.cpp
index 7ed00e336c6..11720d2db67 100644
--- a/src/gallium/drivers/nv50/codegen/nv50_ir_from_sm4.cpp
+++ b/src/gallium/drivers/nv50/codegen/nv50_ir_from_sm4.cpp
@@ -899,10 +899,10 @@ Converter::inspectDeclaration(const sm4_dcl& dcl)
          assert(prog->getType() != Program::TYPE_FRAGMENT);
          break;
       case SM4_SV_CULL_DISTANCE: // XXX: order ?
-         info.io.cullDistanceMask |= 1 << info.io.clipDistanceCount;
-	 // fall through
+         info.io.cullDistanceMask |= 1 << info.io.clipDistanceMask;
+      // fall through
       case SM4_SV_CLIP_DISTANCE:
-         info.io.clipDistanceCount++;
+         info.io.clipDistanceMask++; // abuse as count
          break;
       default:
          break;
@@ -2257,6 +2257,9 @@ Converter::run()
    for (unsigned int pos = 0; pos < sm4.dcls.size(); ++pos)
       handleDeclaration(*sm4.dcls[pos]);
 
+   info.io.genUserClip = -1; // no UCPs permitted with SM4 shaders
+   info.io.clipDistanceMask = (1 << info.io.clipDistanceMask) - 1;
+
    info.assignSlots(&info);
 
    if (sm4.dcls.size() == 0 && sm4.insns.size() == 0)
diff --git a/src/gallium/drivers/nv50/codegen/nv50_ir_from_tgsi.cpp b/src/gallium/drivers/nv50/codegen/nv50_ir_from_tgsi.cpp
index 6cb61b87766..050f2516b99 100644
--- a/src/gallium/drivers/nv50/codegen/nv50_ir_from_tgsi.cpp
+++ b/src/gallium/drivers/nv50/codegen/nv50_ir_from_tgsi.cpp
@@ -593,6 +593,8 @@ public:
 
    bool mainTempsInLMem;
 
+   int clipVertexOutput;
+
    uint8_t *resourceTargets; // TGSI_TEXTURE_*
    unsigned resourceCount;
 
@@ -651,6 +653,8 @@ bool Source::scanSource()
    if (!insns)
       return false;
 
+   clipVertexOutput = -1;
+
    resourceCount = scan.file_max[TGSI_FILE_RESOURCE] + 1;
    resourceTargets = new uint8_t[resourceCount];
 
@@ -707,6 +711,9 @@ bool Source::scanSource()
    if (mainTempsInLMem)
       info->bin.tlsSpace += (scan.file_max[TGSI_FILE_TEMPORARY] + 1) * 16;
 
+   if (info->io.genUserClip > 0)
+      info->io.clipDistanceMask = (1 << info->io.genUserClip) - 1;
+
    return info->assignSlots(info) == 0;
 }
 
@@ -734,6 +741,9 @@ void Source::scanProperty(const struct tgsi_full_property *prop)
    case TGSI_PROPERTY_FS_COORD_PIXEL_CENTER:
       // we don't care
       break;
+   case TGSI_PROPERTY_VS_PROHIBIT_UCPS:
+      info->io.genUserClip = -1;
+      break;
    default:
       INFO("unhandled TGSI property %d\n", prop->Property.PropertyName);
       break;
@@ -820,6 +830,9 @@ bool Source::scanDeclaration(const struct tgsi_full_declaration *decl)
          case TGSI_SEMANTIC_POSITION:
             if (info->type == PIPE_SHADER_FRAGMENT)
                info->io.fragDepth = i;
+            else
+            if (clipVertexOutput < 0)
+               clipVertexOutput = i;
             break;
          case TGSI_SEMANTIC_COLOR:
             if (info->type == PIPE_SHADER_FRAGMENT)
@@ -828,6 +841,14 @@ bool Source::scanDeclaration(const struct tgsi_full_declaration *decl)
          case TGSI_SEMANTIC_EDGEFLAG:
             info->io.edgeFlagOut = i;
             break;
+         case TGSI_SEMANTIC_CLIPVERTEX:
+            clipVertexOutput = i;
+            break;
+         case TGSI_SEMANTIC_CLIPDIST:
+            info->io.clipDistanceMask |=
+               decl->Declaration.UsageMask << (si * 4);
+            info->io.genUserClip = -1;
+            break;
          default:
             break;
          }
@@ -1324,9 +1345,9 @@ Converter::storeDst(int d, int c, Value *val)
    Value *ptr = dst.isIndirect(0) ?
       fetchSrc(dst.getIndirect(0), 0, NULL) : NULL;
 
-   if (info->io.clipDistanceCount &&
+   if (info->io.genUserClip > 0 &&
        dst.getFile() == TGSI_FILE_OUTPUT &&
-       info->out[dst.getIndex(0)].sn == TGSI_SEMANTIC_POSITION) {
+       !dst.isIndirect(0) && dst.getIndex(0) == code->clipVertexOutput) {
       mkMov(clipVtx[c], val);
       val = clipVtx[c];
    }
@@ -2136,7 +2157,7 @@ Converter::handleInstruction(const struct tgsi_full_instruction *insn)
       setPosition(epilogue, true);
       if (prog->getType() == Program::TYPE_FRAGMENT)
          exportOutputs();
-      if (info->io.clipDistanceCount)
+      if (info->io.genUserClip > 0)
          handleUserClipPlanes();
       mkOp(OP_EXIT, TYPE_NONE, NULL)->terminator = 1;
    }
@@ -2173,7 +2194,7 @@ Converter::handleUserClipPlanes()
    int i, c;
 
    for (c = 0; c < 4; ++c) {
-      for (i = 0; i < info->io.clipDistanceCount; ++i) {
+      for (i = 0; i < info->io.genUserClip; ++i) {
          Value *ucp;
          ucp = mkLoad(TYPE_F32, mkSymbol(FILE_MEMORY_CONST, 15, TYPE_F32,
                                          i * 16 + c * 4), NULL);
@@ -2184,7 +2205,7 @@ Converter::handleUserClipPlanes()
       }
    }
 
-   for (i = 0; i < info->io.clipDistanceCount; ++i)
+   for (i = 0; i < info->io.genUserClip; ++i)
       mkOp2(OP_WRSV, TYPE_F32, NULL, mkSysVal(SV_CLIP_DISTANCE, i), res[i]);
 }
 
@@ -2275,7 +2296,7 @@ Converter::run()
    entryBBs.push(entry);
    leaveBBs.push(leave);
 
-   if (info->io.clipDistanceCount) {
+   if (info->io.genUserClip > 0) {
       for (int c = 0; c < 4; ++c)
          clipVtx[c] = getScratch();
    }