From f37b100c34e445736af9128507e6c225e1cc3ec0 Mon Sep 17 00:00:00 2001 From: sluzynsk-amd Date: Tue, 27 Jan 2026 20:51:16 +0100 Subject: [PATCH] SWDEV-563777 - further reduce compilation warnings (#2331) This change resolves some of the warnings generated during clr builds. Quiet regular output of doxygen. Disable non-documented warnings of doxygen. Signed-off-by: Sebastian Luzynski --- projects/clr/hipamd/src/hip_memory.cpp | 2 ++ projects/clr/hipamd/src/hip_mempool.cpp | 3 ++- projects/clr/rocclr/device/device.cpp | 2 +- projects/clr/rocclr/device/rocm/rocvirtual.cpp | 2 +- projects/hip/docs/doxygen-input/doxy.cfg | 4 ++-- 5 files changed, 8 insertions(+), 5 deletions(-) diff --git a/projects/clr/hipamd/src/hip_memory.cpp b/projects/clr/hipamd/src/hip_memory.cpp index f802c4b68c..5cde1a41ca 100644 --- a/projects/clr/hipamd/src/hip_memory.cpp +++ b/projects/clr/hipamd/src/hip_memory.cpp @@ -633,6 +633,8 @@ hipError_t ihipMemcpyCommand(amd::Command*& command, void* dst, const void* src, *srcMemory->asBuffer(), *dstMemory->asBuffer(), sOffset, dOffset, sizeBytes, copyMetadata); break; + case hipHostToHost: + break; } if (command == nullptr) { return hipErrorOutOfMemory; diff --git a/projects/clr/hipamd/src/hip_mempool.cpp b/projects/clr/hipamd/src/hip_mempool.cpp index 92ee5a59fd..afa90c4f0b 100644 --- a/projects/clr/hipamd/src/hip_mempool.cpp +++ b/projects/clr/hipamd/src/hip_mempool.cpp @@ -32,7 +32,8 @@ namespace { inline bool IsMemPoolValid(MemoryPool* mem_pool) { bool result = false; for (auto it : g_devices) { - if (result = it->IsMemoryPoolValid(mem_pool) == true) { + result = it->IsMemoryPoolValid(mem_pool); + if (result) { break; } } diff --git a/projects/clr/rocclr/device/device.cpp b/projects/clr/rocclr/device/device.cpp index e50bdf6517..64e82500d5 100644 --- a/projects/clr/rocclr/device/device.cpp +++ b/projects/clr/rocclr/device/device.cpp @@ -753,7 +753,7 @@ Device::Device() vaCacheAccess_(nullptr), vaCacheMap_(nullptr), index_(0) { - memset(&info_, '\0', sizeof(info_)); + memset(static_cast(&info_), '\0', sizeof(info_)); // By default consider just 1 xcc per device info_.numberOfXccs_ = 1; } diff --git a/projects/clr/rocclr/device/rocm/rocvirtual.cpp b/projects/clr/rocclr/device/rocm/rocvirtual.cpp index c06ded49d0..4f05df6355 100644 --- a/projects/clr/rocclr/device/rocm/rocvirtual.cpp +++ b/projects/clr/rocclr/device/rocm/rocvirtual.cpp @@ -1616,7 +1616,7 @@ void VirtualGPU::dispatchBarrierValuePacket(uint16_t packetHeader, bool resolveD barrier_value_packet_.cond = cond; // Dependent signal and external signal cant be true at the same time - assert(resolveDepSignal & (signal.handle != 0) == 0); + assert((resolveDepSignal && (signal.handle != 0)) == false); if (resolveDepSignal) { auto wait_signals = Barriers().WaitingSignal(); if (wait_signals.size() > 0) { diff --git a/projects/hip/docs/doxygen-input/doxy.cfg b/projects/hip/docs/doxygen-input/doxy.cfg index f4e08a678f..fff25528f7 100644 --- a/projects/hip/docs/doxygen-input/doxy.cfg +++ b/projects/hip/docs/doxygen-input/doxy.cfg @@ -777,7 +777,7 @@ CITE_BIB_FILES = # messages are off. # The default value is: NO. -QUIET = NO +QUIET = YES # The WARNINGS tag can be used to turn on/off the warning messages that are # generated to standard error (stderr) by doxygen. If WARNINGS is set to YES @@ -793,7 +793,7 @@ WARNINGS = YES # will automatically be disabled. # The default value is: YES. -WARN_IF_UNDOCUMENTED = YES +WARN_IF_UNDOCUMENTED = NO # If the WARN_IF_DOC_ERROR tag is set to YES, doxygen will generate warnings for # potential errors in the documentation, such as not documenting some parameters