SpirvShader: Move extension list out of for loop.

Addresses the review comment: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39885/3/src/Pipeline/SpirvShader.cpp#422 Bug: b/145351270 Change-Id: I9a5ceefe048e63e0f52b7e9a15bd84f6bac16629 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39955Tested-by: Ben Clayton <bclayton@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com>

SpirvShader: Move extension list out of for loop.
f6a6a416 · Ben Clayton · 368d39c3 · f6a6a416
Commit f6a6a416 authored Jan 09, 2020 by Ben Clayton
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 3 deletions

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

No files found.
--- a/src/Pipeline/SpirvShader.cpp
+++ b/src/Pipeline/SpirvShader.cpp
@@ -416,12 +416,13 @@ SpirvShader::SpirvShader(

 			case spv::OpExtInstImport:
 			{
+				auto const extensionsByName = std::initializer_list<std::pair<const char *, Extension::Name>>{
+					{ "GLSL.std.450", Extension::GLSLstd450 },
+				};
 				auto id = Extension::ID(insn.word(1));
 				auto name = insn.string(2);
 				auto ext = Extension{ Extension::Unknown };
-				for(auto it : std::initializer_list<std::pair<const char *, Extension::Name>>{
-				        { "GLSL.std.450", Extension::GLSLstd450 },
-				    })
+				for(auto it : extensionsByName)
 				{
 					if(0 == strcmp(name, it.first))
 					{