Commit 4f5b99e3 by Rex Xu

Add more error checks for interpolateAtVertexAMD()

parent c6c80a6e
...@@ -508,6 +508,12 @@ public: ...@@ -508,6 +508,12 @@ public:
return flat || smooth || nopersp; return flat || smooth || nopersp;
#endif #endif
} }
#ifdef AMD_EXTENSIONS
bool isExplicitInterpolation() const
{
return explicitInterp;
}
#endif
bool isAuxiliary() const bool isAuxiliary() const
{ {
return centroid || patch || sample; return centroid || patch || sample;
......
...@@ -1608,6 +1608,23 @@ void TParseContext::builtInOpCheck(const TSourceLoc& loc, const TFunction& fnCan ...@@ -1608,6 +1608,23 @@ void TParseContext::builtInOpCheck(const TSourceLoc& loc, const TFunction& fnCan
if (base == nullptr || base->getType().getQualifier().storage != EvqVaryingIn) if (base == nullptr || base->getType().getQualifier().storage != EvqVaryingIn)
error(loc, "first argument must be an interpolant, or interpolant-array element", fnCandidate.getName().c_str(), ""); error(loc, "first argument must be an interpolant, or interpolant-array element", fnCandidate.getName().c_str(), "");
} }
#ifdef AMD_EXTENSIONS
if (callNode.getOp() == EOpInterpolateAtVertex) {
if (!arg0->getType().getQualifier().isExplicitInterpolation())
error(loc, "argument must be qualified as __explicitInterpAMD in", "interpolant", "");
else {
if (! (*argp)[1]->getAsConstantUnion())
error(loc, "argument must be compile-time constant", "vertex index", "");
else {
unsigned vertexIdx = (*argp)[1]->getAsConstantUnion()->getConstArray()[0].getUConst();
if (vertexIdx > 2)
error(loc, "must be in the range [0, 2]", "vertex index", "");
}
}
}
#endif
break; break;
case EOpEmitStreamVertex: case EOpEmitStreamVertex:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment