Only use '%' as variable name prefix.
authorLei Zhang <antiagainst@google.com>
Fri, 21 Aug 2015 15:52:29 +0000 (11:52 -0400)
committerDejan Mircevski <deki@google.com>
Mon, 24 Aug 2015 19:05:11 +0000 (15:05 -0400)
Since now we can distinguish between def and use according to
the variable's location, there is no need to keep two variable
prefixes.

Also reformat tests to use the value generating instruction
format ("<result-id> = <opcode> <operand>..").

source/binary.cpp
source/text.cpp
test/BinaryToText.cpp
test/NamedId.cpp
test/TextToBinary.cpp
test/ValidateID.cpp

index 918f452..fe48aa9 100644 (file)
@@ -216,7 +216,7 @@ spv_result_t spvBinaryDecodeOperand(
   switch (type) {
     case SPV_OPERAND_TYPE_ID: {
       stream.get() << ((color) ? clr::yellow() : "");
-      stream.get() << "$" << spvFixWord(words[index], endian);
+      stream.get() << "%" << spvFixWord(words[index], endian);
       stream.get() << ((color) ? clr::reset() : "");
       index++;
       position->index++;
index 1c91276..dd4109b 100644 (file)
@@ -339,10 +339,10 @@ spv_result_t spvTextEncodeOperand(
 
   switch (type) {
     case SPV_OPERAND_TYPE_ID: {
-      if ('$' == textValue[0]) {
+      if ('%' == textValue[0]) {
         textValue++;
       }
-      // TODO: Force all ID's to be prefixed with '$'.
+      // TODO: Force all ID's to be prefixed with '%'.
       uint32_t id = 0;
       if (spvTextIsNamedId(textValue)) {
         id = spvNamedIdAssignOrGet(namedIdTable, textValue, pBound);
index 9e2a0c2..702a64e 100644 (file)
@@ -36,25 +36,25 @@ class BinaryToText : public ::testing::Test {
     ASSERT_EQ(SPV_SUCCESS, spvExtInstTableGet(&extInstTable));
 
     const char *textStr = R"(
-OpSource OpenCL 12
-OpMemoryModel Physical64 OpenCL
-OpSourceExtension "PlaceholderExtensionName"
-OpEntryPoint Kernel $1
-OpExecutionMode $1 LocalSizeHint 1 1 1
-OpTypeVoid %2
-OpTypeBool %3
-OpTypeInt %4 8 0
-OpTypeInt %5 8 1
-OpTypeInt %6 16 0
-OpTypeInt %7 16 1
-OpTypeInt %8 32 0
-OpTypeInt %9 32 1
-OpTypeInt %10 64 0
-OpTypeInt %11 64 1
-OpTypeFloat %12 16
-OpTypeFloat %13 32
-OpTypeFloat %14 64
-OpTypeVector %15 4 2
+      OpSource OpenCL 12
+      OpMemoryModel Physical64 OpenCL
+      OpSourceExtension "PlaceholderExtensionName"
+      OpEntryPoint Kernel %1
+      OpExecutionMode %1 LocalSizeHint 1 1 1
+ %2 = OpTypeVoid
+ %3 = OpTypeBool
+ %4 = OpTypeInt 8 0
+ %5 = OpTypeInt 8 1
+ %6 = OpTypeInt 16 0
+ %7 = OpTypeInt 16 1
+ %8 = OpTypeInt 32 0
+ %9 = OpTypeInt 32 1
+%10 = OpTypeInt 64 0
+%11 = OpTypeInt 64 1
+%12 = OpTypeFloat 16
+%13 = OpTypeFloat 32
+%14 = OpTypeFloat 64
+%15 = OpTypeVector 4 2
 )";
     spv_text_t text = {textStr, strlen(textStr)};
     spv_diagnostic diagnostic = nullptr;
@@ -143,15 +143,15 @@ TEST_P(BinaryToTextGLExtSingleFloatInst, Default) {
 OpCapability Shader
 OpExtInstImport %1 "GLSL.std.450"
 OpMemoryModel Logical Simple
-OpEntryPoint Vertex $2 "main"
+OpEntryPoint Vertex %2 "main"
 OpTypeVoid %3
 OpTypeFloat %4 32
-OpConstant $4 %5 1
-OpTypeFunction %6 $3
-OpFunction $3 %2 None $6
+OpConstant %4 %5 1
+OpTypeFunction %6 %3
+OpFunction %3 %2 None %6
 OpLabel %8
-OpExtInst $4 %9 $1 )" + std::string(GetParam().inst) +
-                            R"( $5
+OpExtInst %4 %9 %1 )" + std::string(GetParam().inst) +
+                            R"( %5
 OpReturn
 OpFunctionEnd
 )";
index 3822d64..4a09799 100644 (file)
@@ -30,10 +30,10 @@ TEST(NamedId, Default) {
   const char *spirv = R"(
           OpCapability Shader
           OpMemoryModel Logical Simple
-          OpEntryPoint Vertex $main
+          OpEntryPoint Vertex %main
   %void = OpTypeVoid
-%fnMain = OpTypeFunction $void
-  %main = OpFunction $void None $fnMain
+%fnMain = OpTypeFunction %void
+  %main = OpFunction %void None %fnMain
 %lbMain = OpLabel
           OpReturn
           OpFunctionEnd)";
index 7af8cb0..fcf01dc 100644 (file)
@@ -45,8 +45,8 @@ TEST(TextToBinary, Default) {
       OpSource OpenCL 12
       OpMemoryModel Physical64 OpenCL
       OpSourceExtension "PlaceholderExtensionName"
-      OpEntryPoint Kernel $1
-      OpExecutionMode $1 LocalSizeHint 1 1 1
+      OpEntryPoint Kernel %1
+      OpExecutionMode %1 LocalSizeHint 1 1 1
  %2 = OpTypeVoid
  %3 = OpTypeBool
  ; commment
@@ -286,15 +286,15 @@ TEST_P(GLSingleFloatTest, GLSLExtSingleFloatParamTest) {
             OpCapability Shader
  %glsl450 = OpExtInstImport "GLSL.std.450"
             OpMemoryModel Logical Simple
-            OpEntryPoint Vertex $main "main"
+            OpEntryPoint Vertex %main "main"
     %void = OpTypeVoid
    %float = OpTypeFloat 32
-%const1.5 = OpConstant $float 1.5
-  %fnMain = OpTypeFunction $void
-    %main = OpFunction $void None $fnMain
+%const1.5 = OpConstant %float 1.5
+  %fnMain = OpTypeFunction %void
+    %main = OpFunction %void None %fnMain
   %lbMain = OpLabel
-  %result = OpExtInst $float $glsl450 )" +
-                            std::string(GetParam().inst) + R"( $const1.5
+  %result = OpExtInst %float %glsl450 )" +
+                            std::string(GetParam().inst) + R"( %const1.5
             OpReturn
             OpFunctionEnd
 )";
@@ -357,9 +357,9 @@ TEST_F(TextToBinaryTest, InstructionTwoFormats) {
             OpTypeInt %7 16 1
     %void = OpTypeVoid
             OpTypeFloat %float 32
-%const1.5 = OpConstant $float 1.5
-            OpTypeFunction %fnMain $void
-    %main = OpFunction $void None $fnMain
+%const1.5 = OpConstant %float 1.5
+            OpTypeFunction %fnMain %void
+    %main = OpFunction %void None %fnMain
             OpLabel %lbMain
   %result = OpExtInst $float $glsl450 Round $const1.5
             OpReturn
index 8db6a83..8893385 100644 (file)
@@ -73,112 +73,112 @@ class ValidateID : public ::testing::Test {
 
 TEST_F(ValidateID, OpName) {
   const char *spirv = R"(
-OpName $2 "name"
-OpTypeInt %1 32 0
-OpTypePointer %2 UniformConstant $1
-OpVariable $2 %3 UniformConstant)";
+     OpName %2 "name"
+%1 = OpTypeInt 32 0
+%2 = OpTypePointer UniformConstant %1
+%3 = OpVariable %2 UniformConstant)";
   CHECK(spirv, SPV_SUCCESS);
 }
 
 TEST_F(ValidateID, OpMemberNameGood) {
   const char *spirv = R"(
-OpMemberName $2 0 "foo"
-OpTypeInt %1 32 0
-OpTypeStruct %2 $1)";
+     OpMemberName %2 0 "foo"
+%1 = OpTypeInt 32 0
+%2 = OpTypeStruct %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpMemberNameTypeBad) {
   const char *spirv = R"(
-OpMemberName $1 0 "foo"
-OpTypeInt %1 32 0)";
+     OpMemberName %1 0 "foo"
+%1 = OpTypeInt 32 0)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpMemberNameMemberBad) {
   const char *spirv = R"(
-OpMemberName $2 1 "foo"
-OpTypeInt %1 32 0
-OpTypeStruct %2 $1)";
+     OpMemberName %2 1 "foo"
+%1 = OpTypeInt 32 0
+%2 = OpTypeStruct %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpLineGood) {
   const char *spirv = R"(
-OpString %1 "/path/to/source.file"
-OpLine $4 $1 0 0
-OpTypeInt %2 32 0
-OpTypePointer %3 Generic $2
-OpVariable $3 %4 Generic)";
+%1 = OpString "/path/to/source.file"
+     OpLine %4 %1 0 0
+%2 = OpTypeInt 32 0
+%3 = OpTypePointer Generic %2
+%4 = OpVariable %3 Generic)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpLineFileBad) {
   const char *spirv = R"(
-OpLine $4 $2 0 0
-OpTypeInt %2 32 0
-OpTypePointer %3 Generic $2
-OpVariable $3 %4 Generic)";
+     OpLine %4 %2 0 0
+%2 = OpTypeInt 32 0
+%3 = OpTypePointer Generic %2
+%4 = OpVariable %3 Generic)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpDecorateGood) {
   const char *spirv = R"(
-OpDecorate $2 GLSLShared
-OpTypeInt %1 64 0
-OpTypeStruct %2 $1 $1)";
+     OpDecorate %2 GLSLShared
+%1 = OpTypeInt 64 0
+%2 = OpTypeStruct %1 %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpDecorateBad) {
   const char *spirv = R"(
-OpDecorate $1 GLSLShared)";
+OpDecorate %1 GLSLShared)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpMemberDecorateGood) {
   const char *spirv = R"(
-OpMemberDecorate $2 0 Uniform
-OpTypeInt %1 32 0
-OpTypeStruct %2 $1 $1)";
+     OpMemberDecorate %2 0 Uniform
+%1 = OpTypeInt 32 0
+%2 = OpTypeStruct %1 %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpMemberDecorateBad) {
   const char *spirv = R"(
-OpMemberDecorate $1 0 Uniform
-OpTypeInt %1 32 0)";
+     OpMemberDecorate %1 0 Uniform
+%1 = OpTypeInt 32 0)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpMemberDecorateMemberBad) {
   const char *spirv = R"(
-OpMemberDecorate $2 3 Uniform
-OpTypeInt %1 32 0
-OpTypeStruct %2 $1 $1)";
+     OpMemberDecorate %2 3 Uniform
+%1 = OpTypeInt 32 0
+%2 = OpTypeStruct %1 %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpGroupDecorateGood) {
   const char *spirv = R"(
-OpDecorationGroup %1
-OpDecorate $1 Uniform
-OpDecorate $1 GLSLShared
-OpGroupDecorate $1 $3 $4
-OpTypeInt %2 32 0
-OpConstant $2 %3 42
-OpConstant $2 %4 23)";
+%1 = OpDecorationGroup
+     OpDecorate %1 Uniform
+     OpDecorate %1 GLSLShared
+     OpGroupDecorate %1 %3 %4
+%2 = OpTypeInt 32 0
+%3 = OpConstant %2 42
+%4 = OpConstant %2 23)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpGroupDecorateDecorationGroupBad) {
   const char *spirv = R"(
-OpGroupDecorate $2 $3 $4
-OpTypeInt %2 32 0
-OpConstant $2 %3 42
-OpConstant $2 %4 23)";
+     OpGroupDecorate %2 %3 %4
+%2 = OpTypeInt 32 0
+%3 = OpConstant %2 42
+%4 = OpConstant %2 23)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpGroupDecorateTargetBad) {
   const char *spirv = R"(
-OpDecorationGroup %1
-OpDecorate $1 Uniform
-OpDecorate $1 GLSLShared
-OpGroupDecorate $1 $3
-OpTypeInt %2 32 0)";
+%1 = OpDecorationGroup
+     OpDecorate %1 Uniform
+     OpDecorate %1 GLSLShared
+     OpGroupDecorate %1 %3
+%2 = OpTypeInt 32 0)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
@@ -187,143 +187,143 @@ OpTypeInt %2 32 0)";
 
 TEST_F(ValidateID, OpEntryPointGood) {
   const char *spirv = R"(
-OpEntryPoint GLCompute $3
-OpTypeVoid %1
-OpTypeFunction %2 $1
-OpFunction $1 %3 None $2
-OpLabel %4
-OpReturn
-OpFunctionEnd
+     OpEntryPoint GLCompute %3
+%1 = OpTypeVoid
+%2 = OpTypeFunction %1
+%3 = OpFunction %1 None %2
+%4 = OpLabel
+     OpReturn
+     OpFunctionEnd
 )";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpEntryPointFunctionBad) {
   const char *spirv = R"(
-OpEntryPoint GLCompute $1
-OpTypeVoid %1)";
+     OpEntryPoint GLCompute %1
+%1 = OpTypeVoid)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpEntryPointParameterCountBad) {
   const char *spirv = R"(
-OpEntryPoint GLCompute $3
-OpTypeVoid %1
-OpTypeFunction %2 $1 $1
-OpFunction $1 %3 None $2
-OpLabel %4
-OpReturn
-OpFunctionEnd)";
+     OpEntryPoint GLCompute %3
+%1 = OpTypeVoid
+%2 = OpTypeFunction %1 %1
+%3 = OpFunction %1 None %2
+%4 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpEntryPointReturnTypeBad) {
   const char *spirv = R"(
-OpEntryPoint GLCompute $3
-OpTypeInt %1 32 0
-OpTypeFunction %2 $1
-OpFunction $1 %3 None $2
-OpLabel %4
-OpReturn
-OpFunctionEnd)";
+     OpEntryPoint GLCompute %3
+%1 = OpTypeInt 32 0
+%2 = OpTypeFunction %1
+%3 = OpFunction %1 None %2
+%4 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpExecutionModeGood) {
   const char *spirv = R"(
-OpEntryPoint GLCompute $3
-OpExecutionMode $3 LocalSize 1 1 1
-OpTypeVoid %1
-OpTypeFunction %2 $1
-OpFunction $1 %3 None $2
-OpLabel %4
-OpReturn
-OpFunctionEnd)";
+     OpEntryPoint GLCompute %3
+     OpExecutionMode %3 LocalSize 1 1 1
+%1 = OpTypeVoid
+%2 = OpTypeFunction %1
+%3 = OpFunction %1 None %2
+%4 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpExecutionModeEntryPointBad) {
   const char *spirv = R"(
-OpExecutionMode $3 LocalSize 1 1 1
-OpTypeVoid %1
-OpTypeFunction %2 $1
-OpFunction $1 %3 None $2
-OpLabel %4
-OpReturn
-OpFunctionEnd)";
+     OpExecutionMode %3 LocalSize 1 1 1
+%1 = OpTypeVoid
+%2 = OpTypeFunction %1
+%3 = OpFunction %1 None %2
+%4 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypeVectorGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 4)";
+%1 = OpTypeFloat 32
+%2 = OpTypeVector %1 4)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeVectorComponentTypeBad) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypePointer %2 UniformConstant $1
-OpTypeVector %3 $2 4)";
+%1 = OpTypeFloat 32
+%2 = OpTypePointer UniformConstant %1
+%3 = OpTypeVector %2 4)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypeMatrixGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeVector %2 $1 2
-OpTypeMatrix %3 $2 3)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeVector %1 2
+%3 = OpTypeMatrix %2 3)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeMatrixColumnTypeBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeMatrix %2 $1 3)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeMatrix %1 3)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypeSamplerGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeSampler %2 $1 2D 0 0 0 0)";
+%1 = OpTypeFloat 32
+%2 = OpTypeSampler %1 2D 0 0 0 0)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeSamplerSampledTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeSampler %2 $1 2D 0 0 0 0)";
+%1 = OpTypeVoid
+%2 = OpTypeSampler %1 2D 0 0 0 0)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypeArrayGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 1
-OpTypeArray %3 $1 $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 1
+%3 = OpTypeArray %1 %2)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeArrayElementTypeBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 1
-OpTypeArray %3 $2 $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 1
+%3 = OpTypeArray %2 %2)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpTypeArrayLengthBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 0
-OpTypeArray %3 $1 $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 0
+%3 = OpTypeArray %1 %2)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypeRuntimeArrayGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeRuntimeArray %2 $1)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeRuntimeArray %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeRuntimeArrayBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 0
-OpTypeRuntimeArray %3 $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 0
+%3 = OpTypeRuntimeArray %2)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 // TODO: Object of this type can only be created with OpVariable using the
@@ -331,332 +331,332 @@ OpTypeRuntimeArray %3 $2)";
 
 TEST_F(ValidateID, OpTypeStructGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeFloat %2 64
-OpTypePointer %3 Generic $1
-OpTypeStruct %4 $1 $2 $3)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeFloat 64
+%3 = OpTypePointer Generic %1
+%4 = OpTypeStruct %1 %2 %3)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeStructMemberTypeBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeFloat %2 64
-OpConstant $2 %3 0.0
-OpTypeStruct %4 $1 $2 $3)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeFloat 64
+%3 = OpConstant %2 0.0
+%4 = OpTypeStruct %1 %2 %3)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypePointerGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypePointer %2 Generic $1)";
+%1 = OpTypeInt 32 0
+%2 = OpTypePointer Generic %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypePointerBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 0
-OpTypePointer %3 Generic $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 0
+%3 = OpTypePointer Generic %2)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypeFunctionGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeFunction %2 $1)";
+%1 = OpTypeVoid
+%2 = OpTypeFunction %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypeFunctionReturnTypeBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 0
-OpTypeFunction %3 $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 0
+%3 = OpTypeFunction %2)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpTypeFunctionParameterBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpConstant $2 %3 0
-OpTypeFunction %4 $1 $2 $3)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpConstant %2 0
+%4 = OpTypeFunction %1 %2 %3)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpTypePipeGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 16
-OpTypePipe %3 $2 ReadOnly)";
+%1 = OpTypeFloat 32
+%2 = OpTypeVector %1 16
+%3 = OpTypePipe %2 ReadOnly)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpTypePipeBad) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpConstant $1 %2 0
-OpTypePipe %3 $2 ReadOnly)";
+%1 = OpTypeFloat 32
+%2 = OpConstant %1 0
+%3 = OpTypePipe %2 ReadOnly)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpConstantTrueGood) {
   const char *spirv = R"(
-OpTypeBool %1
-OpConstantTrue $1 %2)";
+%1 = OpTypeBool
+%2 = OpConstantTrue %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantTrueBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpConstantTrue $1 %2)";
+%1 = OpTypeVoid
+%2 = OpConstantTrue %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpConstantFalseGood) {
   const char *spirv = R"(
 OpTypeBool %1
-OpConstantFalse $1 %2)";
+%2 = OpConstantTrue %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantFalseBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpConstantFalse $1 %2)";
+%1 = OpTypeVoid
+%2 = OpConstantFalse %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpConstantGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 1)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpConstant $1 %2 0)";
+%1 = OpTypeVoid
+%2 = OpConstant %1 0)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpConstantCompositeVectorGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 4
-OpConstant $1 %3 3.14
-OpConstantComposite $2 %4 $3 $3 $3 $3)";
+%1 = OpTypeFloat 32
+%2 = OpTypeVector %1 4
+%3 = OpConstant %1 3.14
+%4 = OpConstantComposite %2 %3 %3 %3 %3)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantCompositeVectorResultTypeBad) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 4
-OpConstant $1 %3 3.14
-OpConstantComposite $1 %4 $3 $3 $3 $3)";
+%1 = OpTypeFloat 32
+%2 = OpTypeVector %1 4
+%3 = OpConstant %1 3.14
+%4 = OpConstantComposite %1 %3 %3 %3 %3)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantCompositeVectorConstituentBad) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 4
-OpTypeInt %4 32 0
-OpConstant $1 %3 3.14
-OpConstant $4 %5 42
-OpConstantComposite $2 %6 $3 $5 $3 $3)";
+%1 = OpTypeFloat 32
+%2 = OpTypeVector %1 4
+%4 = OpTypeInt 32 0
+%3 = OpConstant %1 3.14
+%5 = OpConstant %4 42
+%6 = OpConstantComposite %2 %3 %5 %3 %3)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantCompositeMatrixGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 4
-OpTypeMatrix %3 $2 4
-OpConstant $1 %4 1.0
-OpConstant $1 %5 0.0
-OpConstantComposite $2 %6 $4 $5 $5 $5
-OpConstantComposite $2 %7 $5 $4 $5 $5
-OpConstantComposite $2 %8 $5 $5 $4 $5
-OpConstantComposite $2 %9 $5 $5 $5 $4
-OpConstantComposite $3 %10 $6 $7 $8 $9)";
+ %1 = OpTypeFloat 32
+ %2 = OpTypeVector %1 4
+ %3 = OpTypeMatrix %2 4
+ %4 = OpConstant %1 1.0
+ %5 = OpConstant %1 %5 0.0
+ %6 = OpConstantComposite %2 %4 %5 %5 %5
+ %7 = OpConstantComposite %2 %5 %4 %5 %5
+ %8 = OpConstantComposite %2 %5 %5 %4 %5
+ %9 = OpConstantComposite %2 %5 %5 %5 %4
+%10 = OpConstantComposite %3 %6 %7 %8 %9)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantCompositeMatrixConstituentBad) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeVector %2 $1 4
-OpTypeVector %11 $1 3
-OpTypeMatrix %3 $2 4
-OpConstant $1 %4 1.0
-OpConstant $1 %5 0.0
-OpConstantComposite $2 %6 $4 $5 $5 $5
-OpConstantComposite $2 %7 $5 $4 $5 $5
-OpConstantComposite $2 %8 $5 $5 $4 $5
-OpConstantComposite $11 %9 $5 $5 $5
-OpConstantComposite $3 %10 $6 $7 $8 $9)";
+ %1 = OpTypeFloat 32
+ %2 = OpTypeVector %1 4
+%11 = OpTypeVector %1 3
+ %3 = OpTypeMatrix %2 4
+ %4 = OpConstant %1 1.0
+ %5 = OpConstant %1 0.0
+ %6 = OpConstantComposite %2 %4 %5 %5 %5
+ %7 = OpConstantComposite %2 %5 %4 %5 %5
+ %8 = OpConstantComposite %2 %5 %5 %4 %5
+ %9 = OpConstantComposite %11 %5 %5 %5
+%10 = OpConstantComposite %3 %6 %7 %8 %9)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantCompositeMatrixColumnTypeBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeFloat %2 32
-OpTypeVector %3 $1 2
-OpTypeVector %4 $3 2
-OpTypeMatrix %5 $2 2
-OpConstant $1 %6 42
-OpConstant $2 %7 3.14
-OpConstantComposite $3 %8 $6 $6
-OpConstantComposite $4 %9 $7 $7
-OpConstantComposite $5 %10 $8 $9)";
+ %1 = OpTypeInt 32 0
+ %2 = OpTypeFloat 32
+ %3 = OpTypeVector %1 2
+ %4 = OpTypeVector %3 2
+ %5 = OpTypeMatrix %2 2
+ %6 = OpConstant %1 42
+ %7 = OpConstant %2 3.14
+ %8 = OpConstantComposite %3 %6 %6
+ %9 = OpConstantComposite %4 %7 %7
+%10 = OpConstantComposite %5 %8 %9)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantCompositeArrayGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 4
-OpTypeArray %3 $1 $2
-OpConstantComposite $3 %4 $2 $2 $2 $2)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 4
+%3 = OpTypeArray %1 %2
+%4 = OpConstantComposite %3 %2 %2 %2 %2)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantCompositeArrayConstConstituentBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 4
-OpTypeArray %3 $1 $2
-OpConstantComposite $3 %4 $2 $2 $2 $1)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 4
+%3 = OpTypeArray %1 %2
+%4 = OpConstantComposite %3 %2 %2 %2 %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantCompositeArrayConstituentBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpConstant $1 %2 4
-OpTypeArray %3 $1 $2
-OpTypeFloat %5 32
-OpConstant $5 %6 3.14
-OpConstantComposite $3 %4 $2 $2 $2 $6)";
+%1 = OpTypeInt 32 0
+%2 = OpConstant %1 4
+%3 = OpTypeArray %1 %2
+%5 = OpTypeFloat 32
+%6 = OpConstant %5 3.14
+%4 = OpConstantComposite %3 %2 %2 %2 %6)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantCompositeStructGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeInt %2 64 1
-OpTypeStruct %3 $1 $1 $2
-OpConstant $1 %4 42
-OpConstant $2 %5 4300000000
-OpConstantComposite $3 %6 $4 $4 $5)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeInt 64 1
+%3 = OpTypeStruct %1 %1 %2
+%4 = OpConstant %1 42
+%5 = OpConstant %2 4300000000
+%6 = OpConstantComposite %3 %4 %4 %5)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantCompositeStructMemberBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 0
-OpTypeInt %2 64 1
-OpTypeStruct %3 $1 $1 $2
-OpConstant $1 %4 42
-OpConstant $2 %5 4300000000
-OpConstantComposite $3 %6 $4 $5 $4)";
+%1 = OpTypeInt 32 0
+%2 = OpTypeInt 64 1
+%3 = OpTypeStruct %1 %1 %2
+%4 = OpConstant %1 42
+%5 = OpConstant %2 4300000000
+%6 = OpConstantComposite %3 %4 %5 %4)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpConstantSamplerGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpTypeSampler %2 $1 2D 1 0 1 0
-OpConstantSampler $2 %3 ClampToEdge 0 Nearest)";
+%1 = OpTypeFloat 32
+%2 = OpTypeSampler %1 2D 1 0 1 0
+%3 = OpConstantSampler %2 ClampToEdge 0 Nearest)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantSamplerResultTypeBad) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpConstantSampler $1 %2 Clamp 0 Nearest)";
+%1 = OpTypeFloat 32
+%2 = OpConstantSampler %1 Clamp 0 Nearest)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpConstantNullGood) {
   const char *spirv = R"(
-OpTypeBool %1
-OpConstantNull $1 %2
-OpTypeInt %3 32 0
-OpConstantNull $3 %4
-OpTypeFloat %5 32
-OpConstantNull $5 %6
-OpTypePointer %7 UniformConstant $3
-OpConstantNull $7 %8
-OpTypeEvent %9
-OpConstantNull $9 %10
-OpTypeDeviceEvent %11
-OpConstantNull $11 %12
-OpTypeReserveId %13
-OpConstantNull $13 %14
-OpTypeQueue %15
-OpConstantNull $15 %16
-OpTypeVector %17 $3 2
-OpConstantNull $17 %18
-OpTypeMatrix %19 $17 2
-OpConstantNull $19 %20
-OpConstant $3 %25 8
-OpTypeArray %21 $3 $25
-OpConstantNull $21 %22
-OpTypeStruct %23 $3 $5 $1
-OpConstantNull $23 %24
+ %1 = OpTypeBool
+ %2 = OpConstantNull %1
+ %3 = OpTypeInt 32 0
+ %4 = OpConstantNull %3
+ %5 = OpTypeFloat 32
+ %6 = OpConstantNull %5
+ %7 = OpTypePointer UniformConstant %3
+ %8 = OpConstantNull %7
+ %9 = OpTypeEvent
+%10 = OpConstantNull %9
+%11 = OpTypeDeviceEvent
+%12 = OpConstantNull %11
+%13 = OpTypeReserveId
+%14 = OpConstantNull %13
+%15 = OpTypeQueue
+%16 = OpConstantNull %15
+%17 = OpTypeVector %3 2
+%18 = OpConstantNull %17
+%19 = OpTypeMatrix %17 2
+%20 = OpConstantNull %19
+%25 = OpConstant %3 8
+%21 = OpTypeArray %3 %25
+%22 = OpConstantNull %21
+%23 = OpTypeStruct %3 %5 %1
+%24 = OpConstantNull %23
 )";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpConstantNullBasicBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpConstantNull $1 %2)";
+%1 = OpTypeVoid
+%2 = OpConstantNull %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantNullArrayBad) {
   const char *spirv = R"(
-OpTypeInt %1 8 0
-OpTypeInt %2 32 0
-OpTypeSampler %3 $1 2D 0 0 0 0
-OpConstant $2 %4 4
-OpTypeArray %5 $3 $4
-OpConstantNull $5 %6)";
+%1 = OpTypeInt 8 0
+%2 = OpTypeInt 32 0
+%3 = OpTypeSampler %1 2D 0 0 0 0
+%4 = OpConstant %2 4
+%5 = OpTypeArray %3 %4
+%6 = OpConstantNull %5)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpConstantNullStructBad) {
   const char *spirv = R"(
-OpTypeInt %1 8 0
-OpTypeSampler %2 $1 2D 0 0 0 0
-OpTypeStruct %3 $2 $2
-OpConstantNull $3 %4)";
+%1 = OpTypeInt 8 0
+%2 = OpTypeSampler %1 2D 0 0 0 0
+%3 = OpTypeStruct %2 %2
+%4 = OpConstantNull %3)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpSpecConstantTrueGood) {
   const char *spirv = R"(
-OpTypeBool %1
-OpSpecConstantTrue $1 %2)";
+%1 = OpTypeBool
+%2 = OpSpecConstantTrue %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpSpecConstantTrueBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpSpecConstantTrue $1 %2)";
+%1 = OpTypeVoid
+%2 = OpSpecConstantTrue %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpSpecConstantFalseGood) {
   const char *spirv = R"(
-OpTypeBool %1
-OpSpecConstantFalse $1 %2)";
+%1 = OpTypeBool
+%2 = OpSpecConstantFalse %1)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpSpecConstantFalseBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpSpecConstantFalse $1 %2)";
+%1 = OpTypeVoid
+%2 = OpSpecConstantFalse %1)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpSpecConstantGood) {
   const char *spirv = R"(
-OpTypeFloat %1 32
-OpSpecConstant $1 %2 42)";
+%1 = OpTypeFloat 32
+%2 = OpSpecConstant %1 42)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpSpecConstantBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpSpecConstant $1 %2 3.14)";
+%1 = OpTypeVoid
+%2 = OpSpecConstant %1 3.14)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
@@ -665,176 +665,176 @@ OpSpecConstant $1 %2 3.14)";
 
 TEST_F(ValidateID, OpVariableGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 1
-OpTypePointer %2 Generic $1
-OpVariable $2 %3 Generic)";
+%1 = OpTypeInt 32 1
+%2 = OpTypePointer Generic %1
+%3 = OpVariable %2 Generic)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpVariableInitializerGood) {
   const char *spirv = R"(
-OpTypeInt %1 32 1
-OpTypePointer %2 Generic $1
-OpConstant $1 %3 42
-OpVariable $2 %4 Generic $3)";
+%1 = OpTypeInt 32 1
+%2 = OpTypePointer Generic %1
+%3 = OpConstant %1 42
+%4 = OpVariable %2 Generic %3)";
   CHECK(spirv, SPV_SUCCESS);
 }
 // TODO: Positive test OpVariable with OpConstantNull of OpTypePointer
 TEST_F(ValidateID, OpVariableResultTypeBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 1
-OpVariable $1 %2 Generic)";
+%1 = OpTypeInt 32 1
+%2 = OpVariable %1 Generic)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpVariableInitializerBad) {
   const char *spirv = R"(
-OpTypeInt %1 32 1
-OpTypePointer %2 Generic $1
-OpVariable $2 %3 Generic $2)";
+%1 = OpTypeInt 32 1
+%2 = OpTypePointer Generic %1
+%3 = OpVariable %2 Generic %2)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpLoadGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpVariable $3 %5 UniformConstant
-OpFunction $1 %6 None $4
-OpLabel %7
-OpLoad $3 %8 $5
-OpReturn
-OpFunctionEnd
+ %1 = OpTypeVoid
+ %2 = OpTypeInt 32 1
+ %3 = OpTypePointer UniformConstant %2
+ %4 = OpTypeFunction %1
+ %5 = OpVariable %3 UniformConstant
+ %6 = OpFunction %1 None %4
+ %7 = OpLabel
+ %8 = OpLoad %3 %5
+ %9 = OpReturn
+%10 = OpFunctionEnd
 )";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpLoadResultTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpVariable $3 %5 UniformConstant
-OpFunction $1 %6 None $4
-OpLabel %7
-OpLoad $2 %8 $5
-OpReturn
-OpFunctionEnd
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypeFunction %1
+%5 = OpVariable %3 UniformConstant
+%6 = OpFunction %1 None %4
+%7 = OpLabel
+%8 = OpLoad %2 %5
+     OpReturn
+     OpFunctionEnd
 )";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpLoadPointerBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypeFloat %9 32
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpFunction $1 %6 None $4
-OpLabel %7
-OpLoad $9 %8 $3
-OpReturn
-OpFunctionEnd
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%9 = OpTypeFloat 32
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypeFunction %1
+%6 = OpFunction %1 None %4
+%7 = OpLabel
+%8 = OpLoad %9 %3
+     OpReturn
+     OpFunctionEnd
 )";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpStoreGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42
-OpVariable $3 %6 UniformConstant
-OpFunction $1 %7 None $4
-OpLabel %8
-OpStore $6 $5
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42
+%6 = OpVariable %3 UniformConstant
+%7 = OpFunction %1 None %4
+%8 = OpLabel
+     OpStore %6 %5
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpStorePointerBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42
-OpVariable $3 %6 UniformConstant
-OpFunction $1 %7 None $4
-OpLabel %8
-OpStore $3 $5
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42
+%6 = OpVariable %3 UniformConstant
+%7 = OpFunction %1 None %4
+%8 = OpLabel
+     OpStore %3 %5
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpStoreObjectGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42
-OpVariable $3 %6 UniformConstant
-OpFunction $1 %7 None $4
-OpLabel %8
-OpStore $6 $7
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42
+%6 = OpVariable %3 UniformConstant
+%7 = OpFunction %1 None %4
+%8 = OpLabel
+     OpStore %6 %7
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpStoreTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypeFloat %9 32
-OpTypePointer %3 UniformConstant $2
-OpTypeFunction %4 $1
-OpConstant $9 %5 3.14
-OpVariable $3 %6 UniformConstant
-OpFunction $1 %7 None $4
-OpLabel %8
-OpStore $6 $5
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%9 = OpTypeFloat 32
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %9 3.14
+%6 = OpVariable %3 UniformConstant
+%7 = OpFunction %1 None %4
+%8 = OpLabel
+     OpStore %6 %5
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpCopyMemoryGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpConstant $2 %4 42
-OpVariable $3 %5 UniformConstant $4
-OpTypePointer %6 Function $2
-OpTypeFunction %7 $1
-OpFunction $1 %8 None $7
-OpLabel %9
-OpVariable $6 %10 Function
-OpCopyMemory $10 $5 None
-OpReturn
-OpFunctionEnd
+ %1 = OpTypeVoid
+ %2 = OpTypeInt 32 0
+ %3 = OpTypePointer UniformConstant %2
+ %4 = OpConstant %2 42
+ %5 = OpVariable %3 UniformConstant %4
+ %6 = OpTypePointer Function %2
+ %7 = OpTypeFunction %1
+ %8 = OpFunction %1 None %7
+ %9 = OpLabel
+%10 = OpVariable %6 Function
+      OpCopyMemory %10 %5 None
+      OpReturn
+      OpFunctionEnd
 )";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpCopyMemoryBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpConstant $2 %4 42
-OpVariable $3 %5 UniformConstant $4
-OpTypeFloat %11 32
-OpTypePointer %6 Function $11
-OpTypeFunction %7 $1
-OpFunction $1 %8 None $7
-OpLabel %9
-OpVariable $6 %10 Function
-OpCopyMemory $10 $5 None
-OpReturn
-OpFunctionEnd
+ %1 = OpTypeVoid
+ %2 = OpTypeInt 32 0
+ %3 = OpTypePointer UniformConstant %2
+ %4 = OpConstant %2 42
+ %5 = OpVariable %3 UniformConstant %4
+%11 = OpTypeFloat 32
+ %6 = OpTypePointer Function %11
+ %7 = OpTypeFunction %1
+ %8 = OpFunction %1 None %7
+ %9 = OpLabel
+%10 = OpVariable %6 Function
+      OpCopyMemory %10 %5 None
+      OpReturn
+      OpFunctionEnd
 )";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
@@ -842,87 +842,87 @@ OpFunctionEnd
 // TODO: OpCopyMemorySized
 TEST_F(ValidateID, OpCopyMemorySizedGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpTypePointer %4 Function $2
-OpConstant $2 %5 4
-OpVariable $3 %6 UniformConstant $5
-OpTypeFunction %7 $1
-OpFunction $1 %8 None $7
-OpLabel %9
-OpVariable $4 %10 Function
-OpCopyMemorySized $10 $6 $5 None
-OpReturn
-OpFunctionEnd)";
+ %1 = OpTypeVoid
+ %2 = OpTypeInt 32 0
+ %3 = OpTypePointer UniformConstant %2
+ %4 = OpTypePointer Function %2
+ %5 = OpConstant %2 4
+ %6 = OpVariable %3 UniformConstant %5
+ %7 = OpTypeFunction %1
+ %8 = OpFunction %1 None %7
+ %9 = OpLabel
+%10 = OpVariable %4 Function
+      OpCopyMemorySized %10 %6 %5 None
+      OpReturn
+      OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpCopyMemorySizedTargetBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpTypePointer %4 Function $2
-OpConstant $2 %5 4
-OpVariable $3 %6 UniformConstant $5
-OpTypeFunction %7 $1
-OpFunction $1 %8 None $7
-OpLabel %9
-OpCopyMemorySized $9 $6 $5 None
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypePointer Function %2
+%5 = OpConstant %2 4
+%6 = OpVariable %3 UniformConstant %5
+%7 = OpTypeFunction %1
+%8 = OpFunction %1 None %7
+%9 = OpLabel
+     OpCopyMemorySized %9 %6 %5 None
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpCopyMemorySizedSourceBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpTypePointer %4 Function $2
-OpConstant $2 %5 4
-OpTypeFunction %6 $1
-OpFunction $1 %7 None $6
-OpLabel %8
-OpVariable $4 %9 Function
-OpCopyMemorySized $9 $6 $5 None
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypePointer UniformConstant %2
+%4 = OpTypePointer Function %2
+%5 = OpConstant %2 4
+%6 = OpTypeFunction %1
+%7 = OpFunction %1 None %6
+%8 = OpLabel
+%9 = OpVariable %4 Function
+     OpCopyMemorySized %9 %6 %5 None
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpCopyMemorySizedSizeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpTypePointer %4 Function $2
-OpConstant $2 %5 4
-OpVariable $3 %6 UniformConstant $5
-OpTypeFunction %7 $1
-OpFunction $1 %8 None $7
-OpLabel %9
-OpVariable $4 %10 Function
-OpCopyMemorySized $10 $6 $6 None
-OpReturn
-OpFunctionEnd)";
+ %1 = OpTypeVoid
+ %2 = OpTypeInt 32 0
+ %3 = OpTypePointer UniformConstant %2
+ %4 = OpTypePointer Function %2
+ %5 = OpConstant %2 4
+ %6 = OpVariable %3 UniformConstant %5
+ %7 = OpTypeFunction %1
+ %8 = OpFunction %1 None %7
+ %9 = OpLabel
+%10 = OpVariable %4 Function
+      OpCopyMemorySized %10 %6 %6 None
+      OpReturn
+      OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpCopyMemorySizedSizeTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypePointer %3 UniformConstant $2
-OpTypePointer %4 Function $2
-OpConstant $2 %5 4
-OpVariable $3 %6 UniformConstant $5
-OpTypeFunction %7 $1
-OpTypeFloat %11 32
-OpConstant $11 %12 1.0
-OpFunction $1 %8 None $7
-OpLabel %9
-OpVariable $4 %10 Function
-OpCopyMemorySized $10 $6 $12 None
-OpReturn
-OpFunctionEnd)";
+ %1 = OpTypeVoid
+ %2 = OpTypeInt 32 0
+ %3 = OpTypePointer UniformConstant %2
+ %4 = OpTypePointer Function %2
+ %5 = OpConstant %2 4
+ %6 = OpVariable %3 UniformConstant %5
+ %7 = OpTypeFunction %1
+%11 = OpTypeFloat 32
+%12 = OpConstant %11 1.0
+ %8 = OpFunction %1 None %7
+ %9 = OpLabel
+%10 = OpVariable %4 Function
+      OpCopyMemorySized %10 %6 %12 None
+      OpReturn
+      OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
@@ -934,176 +934,176 @@ OpFunctionEnd)";
 
 TEST_F(ValidateID, OpFunctionGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpTypeFunction %3 $1 $2 $2
-OpFunction $1 %4 None $3
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%3 = OpTypeFunction %1 %2 %2
+%4 = OpFunction %1 None %3
+     OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpFunctionResultTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpConstant $2 %5 42
-OpTypeFunction %3 $1 $2 $2
-OpFunction $2 %4 None $3
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%5 = OpConstant %2 42
+%3 = OpTypeFunction %1 %2 %2
+%4 = OpFunction %2 None %3
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpFunctionFunctionTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 1
-OpFunction $1 %4 None $2
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 1
+%4 = OpFunction %1 None %2
 OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpFunctionParameterGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $1 $2
-OpFunction $1 %4 None $3
-OpFunctionParameter $2 %5
-OpLabel %6
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %1 %2
+%4 = OpFunction %1 None %3
+%5 = OpFunctionParameter %2
+%6 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpFunctionParameterResultTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $1 $2
-OpFunction $1 %4 None $3
-OpFunctionParameter $1 %5
-OpLabel %6
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %1 %2
+%4 = OpFunction %1 None %3
+%5 = OpFunctionParameter %1
+%6 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpFunctionParameterOrderBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $1 $2
-OpTypePointer %7 Function $2
-OpFunction $1 %4 None $3
-OpVariable $7 %8 Function
-OpFunctionParameter $2 %5
-OpLabel %6
-OpReturn
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %1 %2
+%7 = OpTypePointer Function %2
+%4 = OpFunction %1 None %3
+%8 = OpVariable %7 Function
+%5 = OpFunctionParameter %2
+%6 = OpLabel
+     OpReturn
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 
 TEST_F(ValidateID, OpFunctionCallGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42 ;21
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42 ;21
 
-OpFunction $2 %6 None $3
-OpFunctionParameter $2 %7
-OpLabel %8
-OpLoad $2 %9 $7
-OpReturnValue $9
-OpFunctionEnd
+%6 = OpFunction %2 None %3
+%7 = OpFunctionParameter %2
+%8 = OpLabel
+%9 = OpLoad %2 %7
+     OpReturnValue %9
+     OpFunctionEnd
 
-OpFunction $1 %10 None $4
-OpLabel %11
-OpReturn
-OpFunctionCall $2 %12 $6 $5
-OpFunctionEnd)";
+%10 = OpFunction %1 None %4
+%11 = OpLabel
+      OpReturn
+%12 = OpFunctionCall %2 %6 %5
+      OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpFunctionCallResultTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42 ;21
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42 ;21
 
-OpFunction $2 %6 None $3
-OpFunctionParameter $2 %7
-OpLabel %8
-OpLoad $2 %9 $7
-OpReturnValue $9
-OpFunctionEnd
+%6 = OpFunction %2 None %3
+%7 = OpFunctionParameter %2
+%8 = OpLabel
+%9 = OpLoad %2 %7
+     OpReturnValue %9
+     OpFunctionEnd
 
-OpFunction $1 %10 None $4
-OpLabel %11
-OpReturn
-OpFunctionCall $1 %12 $6 $5
-OpFunctionEnd)";
+%10 = OpFunction %1 None %4
+%11 = OpLabel
+      OpReturn
+%12 = OpFunctionCall %1 %6 %5
+      OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpFunctionCallFunctionBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42 ;21
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42 ;21
 
-OpFunction $1 %10 None $4
-OpLabel %11
-OpReturn
-OpFunctionCall $2 %12 $5 $5
-OpFunctionEnd)";
+%10 = OpFunction %1 None %4
+%11 = OpLabel
+      OpReturn
+%12 = OpFunctionCall %2 %5 %5
+      OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 TEST_F(ValidateID, OpFunctionCallArgumentTypeBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42
 
-OpTypeFloat %13 32
-OpConstant $13 %14 3.14
+%13 = OpTypeFloat 32
+%14 = OpConstant %13 3.14
 
-OpFunction $2 %6 None $3
-OpFunctionParameter $2 %7
-OpLabel %8
-OpLoad $2 %9 $7
-OpReturnValue $9
-OpFunctionEnd
+%6 = OpFunction %2 None %3
+%7 = OpFunctionParameter %2
+%8 = OpLabel
+%9 = OpLoad %2 %7
+     OpReturnValue %9
+     OpFunctionEnd
 
-OpFunction $1 %10 None $4
-OpLabel %11
-OpReturn
-OpFunctionCall $2 %12 $6 $14
-OpFunctionEnd)";
+%10 = OpFunction %1 None %4
+%11 = OpLabel
+      OpReturn
+%12 = OpFunctionCall %2 %6 %14
+      OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 #if 0
 TEST_F(ValidateID, OpFunctionCallArgumentCountBar) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpTypeFunction %4 $1
-OpConstant $2 %5 42 ;21
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpTypeFunction %1
+%5 = OpConstant %2 42 ;21
 
-OpFunction $2 %6 None $3
-OpFunctionParameter $2 %7
-OpLabel %8
-OpLoad $2 %9 $7
-OpReturnValue $9
-OpFunctionEnd
+%6 = OpFunction %2 None %3
+%7 = OpFunctionParameter %2
+%8 = OpLabel
+%9 = OpLoad %2 %7
+     OpReturnValue %9
+     OpFunctionEnd
 
-OpFunction $1 %10 None $4
-OpLabel %11
-OpReturn
-OpFunctionCall $2 %12 $6 $5
-OpFunctionEnd)";
+%10 = OpFunction %1 None %4
+%11 = OpLabel
+      OpReturn
+%12 = OpFunctionCall %2 %6 %5
+      OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }
 #endif
@@ -1240,40 +1240,40 @@ OpFunctionEnd)";
 
 TEST_F(ValidateID, OpReturnValueConstantGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpConstant $2 %4 42
-OpFunction $2 %5 None $3
-OpLabel %6
-OpReturnValue $4
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpConstant %2 42
+%5 = OpFunction %2 None %3
+%6 = OpLabel
+     OpReturnValue %4
+     OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpReturnValueVariableGood) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0 ;10
-OpTypeFunction %3 $2 $2 ;14
-OpTypePointer %8 Function $2 ;18
-OpConstant $2 %4 42 ;22
-OpFunction $2 %5 None $3 ;27
-OpLabel %6 ;29
-OpVariable $8 %7 Function $4 ;34
-OpReturnValue $7 ;36
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0 ;10
+%3 = OpTypeFunction %2 %2 ;14
+%8 = OpTypePointer Function %2 ;18
+%4 = OpConstant %2 42 ;22
+%5 = OpFunction %2 None %3 ;27
+%6 = OpLabel ;29
+%7 = OpVariable %8 Function %4 ;34
+     OpReturnValue %7 ;36
+     OpFunctionEnd)";
   CHECK(spirv, SPV_SUCCESS);
 }
 TEST_F(ValidateID, OpReturnValueBad) {
   const char *spirv = R"(
-OpTypeVoid %1
-OpTypeInt %2 32 0
-OpTypeFunction %3 $2 $2
-OpConstant $2 %4 42
-OpFunction $2 %5 None $3
-OpLabel %6
-OpReturnValue $1
-OpFunctionEnd)";
+%1 = OpTypeVoid
+%2 = OpTypeInt 32 0
+%3 = OpTypeFunction %2 %2
+%4 = OpConstant %2 42
+%5 = OpFunction %2 None %3
+%6 = OpLabel
+     OpReturnValue %1
+     OpFunctionEnd)";
   CHECK(spirv, SPV_ERROR_INVALID_ID);
 }