Skip to content

Commit 79c4baf

Browse files
authored
Merge pull request #2604 from zhaomaosu/asan-avoid-direct-ur-call
[DevASAN] Avoid directly UR api call
2 parents be058f4 + 8a13814 commit 79c4baf

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

source/loader/layers/sanitizer/asan/asan_shadow.hpp

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515

1616
#include "asan_allocator.hpp"
1717
#include "sanitizer_common/sanitizer_libdevice.hpp"
18+
#include "ur_sanitizer_layer.hpp"
1819

1920
#include <unordered_set>
2021

@@ -24,10 +25,16 @@ namespace asan {
2425
struct ShadowMemory {
2526
ShadowMemory(ur_context_handle_t Context, ur_device_handle_t Device)
2627
: Context(Context), Device(Device) {
27-
urDeviceRetain(Device);
28+
[[maybe_unused]] ur_result_t URes =
29+
getContext()->urDdiTable.Device.pfnRetain(Device);
30+
assert(URes == UR_RESULT_SUCCESS);
2831
}
2932

30-
virtual ~ShadowMemory() { urDeviceRelease(Device); }
33+
virtual ~ShadowMemory() {
34+
[[maybe_unused]] ur_result_t URes =
35+
getContext()->urDdiTable.Device.pfnRelease(Device);
36+
assert(URes == UR_RESULT_SUCCESS);
37+
}
3138

3239
virtual ur_result_t Setup() = 0;
3340

0 commit comments

Comments
 (0)