commit 3954a0bafb8aba97b8fdc365a39161f896edc199
author Ben Clayton <bclayton@google.com> Wed Apr 10 14:56:20 2019 -0400
committer Ben Clayton <bclayton@google.com> Thu Apr 11 03:50:19 2019 +0000
tree 0c9d30388d9fdd5ce793c9e586861d1b20f181a8
parent 3f007c4d547314102616902478cea80cfa98e586

SpirvShader: Document ops requiring capabilities

... that will not be required for Pastel 1.0.

Bug: b/126873455
Change-Id: I024296dd39ae74613577bd5dbd8d75051b7f4f5c
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28790
Tested-by: Ben Clayton <bclayton@google.com>
Presubmit-Ready: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

src/Pipeline/SpirvShader.cpp

diff --git a/src/Pipeline/SpirvShader.cpp b/src/Pipeline/SpirvShader.cpp
index 8578389..e0bcdff 100644
--- a/src/Pipeline/SpirvShader.cpp
+++ b/src/Pipeline/SpirvShader.cpp
@@ -3595,11 +3595,13 @@
 		}
 		case GLSLstd450PackDouble2x32:
 		{
+			// Requires Float64 capability.
 			UNIMPLEMENTED("GLSLstd450PackDouble2x32");
 			break;
 		}
 		case GLSLstd450UnpackDouble2x32:
 		{
+			// Requires Float64 capability.
 			UNIMPLEMENTED("GLSLstd450UnpackDouble2x32");
 			break;
 		}
@@ -3634,16 +3636,19 @@
 		}
 		case GLSLstd450InterpolateAtCentroid:
 		{
+			// Requires sampleRateShading / InterpolationFunction capability.
 			UNIMPLEMENTED("GLSLstd450InterpolateAtCentroid");
 			break;
 		}
 		case GLSLstd450InterpolateAtSample:
 		{
+			// Requires sampleRateShading / InterpolationFunction capability.
 			UNIMPLEMENTED("GLSLstd450InterpolateAtSample");
 			break;
 		}
 		case GLSLstd450InterpolateAtOffset:
 		{
+			// Requires sampleRateShading / InterpolationFunction capability.
 			UNIMPLEMENTED("GLSLstd450InterpolateAtOffset");
 			break;
 		}