Comment out some dead assignments per suggestion from static analysis tools.

Code is left in as comments for clarity to humans.

Comment out some dead assignments per suggestion from static analysis tools.
465a1466 · Steve · a417f010 · 465a1466
Commit 465a1466 authored Jul 15, 2015 by Steve
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 6 deletions

SPVRemapper.cpp SPIRV/SPVRemapper.cpp +7 -6

No files found.
--- a/SPIRV/SPVRemapper.cpp
+++ b/SPIRV/SPVRemapper.cpp
@@ -450,11 +450,12 @@ namespace spv {
                return nextInst;

            case spv::OperandVariableLiterals:
-                if (opCode == spv::OpDecorate && asDecoration(word - 1) == spv::DecorationBuiltIn) {
-                    ++word;
-                    --numOperands;
-                }
-                word += numOperands;
+                // for clarity
+                // if (opCode == spv::OpDecorate && asDecoration(word - 1) == spv::DecorationBuiltIn) {
+                //     ++word;
+                //     --numOperands;
+                // }
+                // word += numOperands;
                return nextInst;

            case spv::OperandVariableLiteralId:
@@ -466,7 +467,7 @@ namespace spv {
                return nextInst;

            case spv::OperandLiteralString:
-                word += literalStringWords(literalString(word));
+                // word += literalStringWords(literalString(word)); // for clarity
                return nextInst;

                // Single word operands we simply ignore, as they hold no IDs