address comments

Author: AllanZyne

source/loader/layers/sanitizer/asan_interceptor.cpp

@@ -187,6 +187,8 @@ SanitizerInterceptor::~SanitizerInterceptor() {
     DestroyShadowMemoryOnPVC();
     DestroyShadowMemoryOnDG2();
 
+    // We must release these objects before releasing adapters, since
+    // they may use the adapter in their destruction process
     m_Quarantine = nullptr;
     m_MemBufferMap.clear();
     m_AllocationMap.clear();

source/loader/layers/sanitizer/asan_interceptor.hpp

@@ -49,9 +49,8 @@ struct DeviceInfo {
     std::queue<std::shared_ptr<AllocInfo>> Quarantine;
     size_t QuarantineSize = 0;
 
-    // TODO: re-enable retaining and releasing device handles in DeviceInfo
-    // constructor/destructor. See PR
-    // https://github.com/oneapi-src/unified-runtime/pull/1883
+    // Device handles are special and alive in the whole process lifetime,
+    // so we needn't retain&release here.
     explicit DeviceInfo(ur_device_handle_t Device) : Handle(Device) {}
 
     ur_result_t allocShadowMemory(ur_context_handle_t Context);

test/layers/sanitizer/asan.cpp

@@ -6,7 +6,7 @@
  * See LICENSE.TXT
  * SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
  *
- * @file codeloc.cpp
+ * @file asan.cpp
  *
  */