Add support for OpInBoundsAccessChain

This can be treated exactly the same as OpAccessChain. Bug: b/128690261 Bug: b/127950082 Test: dEQP-VK.spirv_assembly.instruction.compute.opinboundsaccesschain.* Change-Id: I65312ad48e461ec83e6999c6a899820e91e2a68b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27368Tested-by: Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

Add support for OpInBoundsAccessChain
10fd6247 · Chris Forbes · f0d22ad9 · 10fd6247
Commit 10fd6247 authored Mar 15, 2019 by Chris Forbes
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

SpirvShader.cpp src/Pipeline/SpirvShader.cpp +3 -1

No files found.
--- a/src/Pipeline/SpirvShader.cpp
+++ b/src/Pipeline/SpirvShader.cpp
@@ -308,6 +308,7 @@ namespace sw

 			case spv::OpLoad:
 			case spv::OpAccessChain:
+			case spv::OpInBoundsAccessChain:
 			case spv::OpCompositeConstruct:
 			case spv::OpCompositeInsert:
 			case spv::OpCompositeExtract:
@@ -386,7 +387,7 @@ namespace sw
 				object.kind = Object::Kind::Value;
 				object.definition = insn;

-				if (insn.opcode() == spv::OpAccessChain)
+				if (insn.opcode() == spv::OpAccessChain || insn.opcode() == spv::OpInBoundsAccessChain)
 				{
 					// interior ptr has two parts:
 					// - logical base ptr, common across all lanes and known at compile time
@@ -1083,6 +1084,7 @@ namespace sw
 			break;

 		case spv::OpAccessChain:
+		case spv::OpInBoundsAccessChain:
 			EmitAccessChain(insn, routine);
 			break;