Consistently use `const auto` It reads more naturally compared with `auto const`, and there are more existing instances of it too. This change `sed`s the repo to turn the latter into the former for consistency. Bug: b/245568070 Change-Id: Iffef96f49239e06d9eaa5ff309c50bf5bb0bdbb3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/68189 Tested-by: Shahbaz Youssefi <syoussefi@google.com> Commit-Queue: Shahbaz Youssefi <syoussefi@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com> Kokoro-Result: kokoro <noreply+kokoro@google.com> Presubmit-Ready: Shahbaz Youssefi <syoussefi@google.com>
diff --git a/src/Pipeline/SpirvShader.cpp b/src/Pipeline/SpirvShader.cpp index ee3e1a5..2357a52 100644 --- a/src/Pipeline/SpirvShader.cpp +++ b/src/Pipeline/SpirvShader.cpp
@@ -196,8 +196,8 @@ case spv::OpGroupDecorate: { uint32_t group = insn.word(1); - auto const &groupDecorations = decorations[group]; - auto const &descriptorGroupDecorations = descriptorDecorations[group]; + const auto &groupDecorations = decorations[group]; + const auto &descriptorGroupDecorations = descriptorDecorations[group]; for(auto i = 2u; i < insn.wordCount(); i++) { // Remaining operands are targets to apply the group to. @@ -210,7 +210,7 @@ case spv::OpGroupMemberDecorate: { - auto const &srcDecorations = decorations[insn.word(1)]; + const auto &srcDecorations = decorations[insn.word(1)]; for(auto i = 2u; i < insn.wordCount(); i += 2) { // remaining operands are pairs of <id>, literal for members to apply to. @@ -1165,7 +1165,7 @@ ApplyDecorationsForId(&d, id); - auto const &obj = getType(id); + const auto &obj = getType(id); switch(obj.opcode()) { case spv::OpTypePointer: