commit f3a430d1ae8f5a4b27909d25d01c03be5030ddb1
author Chris Forbes <chrisforbes@google.com> Fri Mar 08 07:51:39 2019 -0800
committer Chris Forbes <chrisforbes@google.com> Fri Mar 08 16:56:55 2019 +0000
tree b5171067b244b2068c08c3d31bc8eea5412c2fbc
parent abdfa17a3b4aa9eaaaa107396d97dd034c6e124d

Skip remaining debug instructions

Change-Id: Ie4e2c11c495f17bf628b3c828ce81596e3d73e5f
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26588
Reviewed-by: Ben Clayton <bclayton@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: 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 2f05ded..163b20f 100644
--- a/src/Pipeline/SpirvShader.cpp
+++ b/src/Pipeline/SpirvShader.cpp
@@ -246,6 +246,10 @@
 			case spv::OpSource:
 			case spv::OpSourceContinued:
 			case spv::OpSourceExtension:
+			case spv::OpLine:
+			case spv::OpNoLine:
+			case spv::OpModuleProcessed:
+			case spv::OpString:
 				// No semantic impact
 				break;
 
@@ -952,6 +956,10 @@
 			case spv::OpSource:
 			case spv::OpSourceContinued:
 			case spv::OpSourceExtension:
+			case spv::OpLine:
+			case spv::OpNoLine:
+			case spv::OpModuleProcessed:
+			case spv::OpString:
 				// Nothing to do at emit time. These are either fully handled at analysis time,
 				// or don't require any work at all.
 				break;