Merge pull request #1296 from kbenzie/benie/asan-move-not-copy

kbenzie · web-flow · commit 5d5c8106cf34 · 2024-01-31T11:19:01.000Z
[ASAN] Fix copy instead of move
diff --git a/source/loader/layers/sanitizer/asan_interceptor.cpp b/source/loader/layers/sanitizer/asan_interceptor.cpp
@@ -149,7 +149,7 @@ ur_result_t SanitizerInterceptor::allocateMemory(
     // For memory release
     {
         std::scoped_lock<ur_shared_mutex> Guard(ContextInfo->Mutex);
-        ContextInfo->AllocatedUSMMap[AllocBegin] = AllocInfo;
+        ContextInfo->AllocatedUSMMap[AllocBegin] = std::move(AllocInfo);
     }
 
     context.logger.info(

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ ur_result_t SanitizerInterceptor::allocateMemory(`
`149`	`149`	`// For memory release`
`150`	`150`	`{`
`151`	`151`	`std::scoped_lock<ur_shared_mutex> Guard(ContextInfo->Mutex);`
`152`		`- ContextInfo->AllocatedUSMMap[AllocBegin] = AllocInfo;`
	`152`	`+ ContextInfo->AllocatedUSMMap[AllocBegin] = std::move(AllocInfo);`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`context.logger.info(`