commit 185189caade33cbe672c1cf018e94c5536bb11b6
author Jonah Ryan-Davis <8729214+jonahryandavis@users.noreply.github.com> Tue Jul 14 10:28:52 2020 -0400
committer John Zulauf <32470354+jzulauf-lunarg@users.noreply.github.com> Tue Jul 14 16:53:19 2020 -0600
tree 277c94a115320a0430037a6dea20ac91316220fc
parent aa97d8b8d42f901f48f4d069179599782c90761a

Fix -Wrange-loop-construct range var copy warning

Change range loop variable (and subsequent field reference) to
references to avoid copy (and copy warning).

Change-Id: I81255f3c0c787256bb208de0581a375b3b12b383

layers/synchronization_validation.cpp

diff --git a/layers/synchronization_validation.cpp b/layers/synchronization_validation.cpp
index e51c5d1..854bff3 100644
--- a/layers/synchronization_validation.cpp
+++ b/layers/synchronization_validation.cpp
@@ -170,8 +170,8 @@
                                            std::map<VkPipelineStageFlagBits, VkPipelineStageFlags> &map) {
     VkPipelineStageFlags unscanned = stage_mask;
     VkPipelineStageFlags related = 0;
-    for (const auto entry : map) {
-        const auto stage = entry.first;
+    for (const auto &entry : map) {
+        const auto &stage = entry.first;
         if (stage & unscanned) {
             related = related | entry.second;
             unscanned = unscanned & ~stage;