Skip to content

Commit 86cb7fc

Browse files
authored
Fix minor performance issues with variable copy (#394)
Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
1 parent e64ee71 commit 86cb7fc

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

source/layers/validation/checkers/basic_leak/zel_basic_leak_checker.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,7 @@ namespace validation_layer
309309
std::cerr << " \\--->";
310310

311311
for (size_t i = 0; i < dtors.size(); i++) {
312-
auto name = dtors[i];
312+
const auto& name = dtors[i];
313313
auto zeCount = counts[name].load();
314314
diff -= zeCount;
315315

source/loader/ze_loader.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ namespace loader
235235
}
236236

237237
// Replace the original driver vector with the ordered one
238-
*drivers = orderedDrivers;
238+
*drivers = std::move(orderedDrivers);
239239

240240
if (debugTraceEnabled) {
241241
std::string message = "driverOrdering: Drivers after ZEL_DRIVERS_ORDER:";
@@ -676,7 +676,7 @@ namespace loader
676676
}
677677
}
678678

679-
for( auto driverInfo : discoveredDrivers )
679+
for( const auto& driverInfo : discoveredDrivers )
680680
{
681681
if (discoveredDrivers.size() == 1) {
682682
auto handle = LOAD_DRIVER_LIBRARY( driverInfo.path.c_str() );

0 commit comments

Comments
 (0)