New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move away from Memory management APIs deprecated in ROCM 6.3 #71

Open

xinyazhang wants to merge 2 commits into rocm6.3_internal_testing from xinyazhang/SWDEV-491405

+17 −9

Open

Move away from Memory management APIs deprecated in ROCM 6.3 #71

Move away from Memory management APIs deprecated in ROCM 6.3

GitHub Actions / cpplint completed Oct 15, 2024 in 0s

reviewdog [cpplint] report

reported by reviewdog 🐶

Findings (2)

onnxruntime/core/providers/migraphx/migraphx_allocator.cc|75| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
onnxruntime/core/providers/rocm/rocm_allocator.cc|93| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]

Filtered Findings (0)

Annotations

Check warning on line 75 in onnxruntime/core/providers/migraphx/migraphx_allocator.cc

github-actions / cpplint

[cpplint] onnxruntime/core/providers/migraphx/migraphx_allocator.cc#L75

Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]

Raw output


            onnxruntime/core/providers/migraphx/migraphx_allocator.cc:75:  Using C-style cast.  Use reinterpret_cast<void**>(...) instead  [readability/casting] [4]

Check warning on line 93 in onnxruntime/core/providers/rocm/rocm_allocator.cc

github-actions / cpplint

[cpplint] onnxruntime/core/providers/rocm/rocm_allocator.cc#L93

Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]

Raw output


            onnxruntime/core/providers/rocm/rocm_allocator.cc:93:  Using C-style cast.  Use reinterpret_cast<void**>(...) instead  [readability/casting] [4]

View more details on GitHub Actions