Warning Fixes

This commit is contained in:
RipleyTom 2022-03-23 11:26:24 +01:00 committed by Megamouse
parent af0e1f609e
commit a4d715e25d
12 changed files with 34 additions and 37 deletions

View file

@ -90,7 +90,7 @@ void FAudioBackend::Pause()
FAudio_.error("FAudioSourceVoice_Stop() failed(0x%08x)", res); FAudio_.error("FAudioSourceVoice_Stop() failed(0x%08x)", res);
} }
if (res = FAudioSourceVoice_FlushSourceBuffers(m_source_voice)) if ((res = FAudioSourceVoice_FlushSourceBuffers(m_source_voice)))
{ {
FAudio_.error("FAudioSourceVoice_FlushSourceBuffers() failed(0x%08x)", res); FAudio_.error("FAudioSourceVoice_FlushSourceBuffers() failed(0x%08x)", res);
} }

View file

@ -967,7 +967,7 @@ error_code cellGemGetCameraState(vm::ptr<CellGemCameraState> camera_state)
{ {
cellGem.todo("cellGemGetCameraState(camera_state=0x%x)", camera_state); cellGem.todo("cellGemGetCameraState(camera_state=0x%x)", camera_state);
auto& gem = g_fxo->get<gem_config>(); [[maybe_unused]] auto& gem = g_fxo->get<gem_config>();
if (!camera_state) if (!camera_state)
{ {

View file

@ -1642,7 +1642,7 @@ s32 cellSpursFinalize(vm::ptr<CellSpurs> spurs)
return CELL_SPURS_CORE_ERROR_STAT; return CELL_SPURS_CORE_ERROR_STAT;
} }
u32 wklEnabled = spurs->wklEnabled.load(); [[maybe_unused]] u32 wklEnabled = spurs->wklEnabled.load();
if (spurs->flags1 & SF1_32_WORKLOADS) if (spurs->flags1 & SF1_32_WORKLOADS)
{ {

View file

@ -1602,7 +1602,7 @@ spu_thread::spu_thread(lv2_spu_group* group, u32 index, std::string_view name, u
else else
{ {
// alloc_hidden indicates falloc to allocate page with no access rights in base memory // alloc_hidden indicates falloc to allocate page with no access rights in base memory
ensure(vm::get(vm::spu)->falloc(vm_offset(), SPU_LS_SIZE, &shm, vm::page_size_64k | vm::alloc_hidden)); ensure(vm::get(vm::spu)->falloc(vm_offset(), SPU_LS_SIZE, &shm, static_cast<u64>(vm::page_size_64k) | static_cast<u64>(vm::alloc_hidden)));
} }
// Try to guess free area // Try to guess free area

View file

@ -88,7 +88,9 @@ error_code sys_tty_read(s32 ch, vm::ptr<char> buf, u32 len, vm::ptr<u32> preadle
return CELL_OK; return CELL_OK;
} }
error_code sys_tty_write(ppu_thread& ppu, s32 ch, vm::cptr<char> buf, u32 len, vm::ptr<u32> pwritelen) std::string dump_useful_thread_info();
error_code sys_tty_write([[maybe_unused]] ppu_thread& ppu, s32 ch, vm::cptr<char> buf, u32 len, vm::ptr<u32> pwritelen)
{ {
sys_tty.notice("sys_tty_write(ch=%d, buf=*0x%x, len=%d, pwritelen=*0x%x)", ch, buf, len, pwritelen); sys_tty.notice("sys_tty_write(ch=%d, buf=*0x%x, len=%d, pwritelen=*0x%x)", ch, buf, len, pwritelen);
@ -113,7 +115,6 @@ error_code sys_tty_write(ppu_thread& ppu, s32 ch, vm::cptr<char> buf, u32 len, v
return current - last_write.exchange(current) >= 500'000; return current - last_write.exchange(current) >= 500'000;
}()) }())
{ {
std::string dump_useful_thread_info();
ppu_log.notice("\n%s", dump_useful_thread_info()); ppu_log.notice("\n%s", dump_useful_thread_info());
} }

View file

@ -978,12 +978,12 @@ private:
return commit_param(rsxaudio_port, avport_idx, avport_src, freq, bit_cnt, spdif_use_serial_buf, pkt.audio_cs_info); return commit_param(rsxaudio_port, avport_idx, avport_src, freq, bit_cnt, spdif_use_serial_buf, pkt.audio_cs_info);
} }
bool commit_param(u8 rsxaudio_port, RsxaudioAvportIdx avport, u8 avport_src, u32 freq, bool commit_param(u8 rsxaudio_port, RsxaudioAvportIdx avport, [[maybe_unused]] u8 avport_src, [[maybe_unused]] u32 freq,
UartAudioSampleSize bit_cnt, bool spdif_use_serial_buf, const u8 *cs_data) UartAudioSampleSize bit_cnt, [[maybe_unused]] bool spdif_use_serial_buf, [[maybe_unused]] const u8 *cs_data)
{ {
// TBA // TBA
const auto avport_idx = static_cast<std::underlying_type_t<decltype(avport)>>(avport); [[maybe_unused]] const auto avport_idx = static_cast<std::underlying_type_t<decltype(avport)>>(avport);
const auto rsxaudio_word_depth = bit_cnt == UartAudioSampleSize::_16BIT ? RsxaudioSampleSize::_16BIT : RsxaudioSampleSize::_32BIT; [[maybe_unused]] const auto rsxaudio_word_depth = bit_cnt == UartAudioSampleSize::_16BIT ? RsxaudioSampleSize::_16BIT : RsxaudioSampleSize::_32BIT;
switch (rsxaudio_port) switch (rsxaudio_port)
{ {
@ -995,7 +995,7 @@ private:
case SYS_RSXAUDIO_PORT_SPDIF_0: case SYS_RSXAUDIO_PORT_SPDIF_0:
case SYS_RSXAUDIO_PORT_SPDIF_1: case SYS_RSXAUDIO_PORT_SPDIF_1:
{ {
const u8 spdif_idx = rsxaudio_port == SYS_RSXAUDIO_PORT_SPDIF_1; [[maybe_unused]] const u8 spdif_idx = rsxaudio_port == SYS_RSXAUDIO_PORT_SPDIF_1;
// TBA // TBA
break; break;
@ -1067,7 +1067,7 @@ struct audio_set_active_cmd : public ps3av_cmd
return; return;
} }
const bool requested_avports[SYS_RSXAUDIO_AVPORT_CNT] = [[maybe_unused]] const bool requested_avports[SYS_RSXAUDIO_AVPORT_CNT] =
{ {
(pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_0) != 0U, (pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_0) != 0U,
(pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_1) != 0U, (pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_1) != 0U,
@ -1122,7 +1122,7 @@ struct audio_spdif_bit_cmd : public ps3av_cmd
return; return;
} }
const bool requested_avports[SYS_RSXAUDIO_AVPORT_CNT] = [[maybe_unused]] const bool requested_avports[SYS_RSXAUDIO_AVPORT_CNT] =
{ {
(pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_0) != 0U, (pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_0) != 0U,
(pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_1) != 0U, (pkt->audio_port & PS3AV_AUDIO_PORT_HDMI_1) != 0U,
@ -1301,7 +1301,7 @@ struct inc_avset_cmd : public ps3av_cmd
break; break;
} }
const u8 hdmi_idx = av_audio_pkt->avport == static_cast<u16>(UartAudioAvport::HDMI_1); [[maybe_unused]] const u8 hdmi_idx = av_audio_pkt->avport == static_cast<u16>(UartAudioAvport::HDMI_1);
// TBA // TBA
} }

View file

@ -56,35 +56,35 @@ SSE4_1_FUNC static inline u16 sse41_hmax_epu16(__m128i x)
} }
#if defined(__AVX512F__) && defined(__AVX512VL__) && defined(__AVX512DQ__) && defined(__AVX512CD__) && defined(__AVX512BW__) #if defined(__AVX512F__) && defined(__AVX512VL__) && defined(__AVX512DQ__) && defined(__AVX512CD__) && defined(__AVX512BW__)
constexpr bool s_use_ssse3 = true; [[maybe_unused]] constexpr bool s_use_ssse3 = true;
constexpr bool s_use_sse4_1 = true; constexpr bool s_use_sse4_1 = true;
constexpr bool s_use_avx2 = true; constexpr bool s_use_avx2 = true;
constexpr bool s_use_avx3 = true; [[maybe_unused]] constexpr bool s_use_avx3 = true;
#elif defined(__AVX2__) #elif defined(__AVX2__)
constexpr bool s_use_ssse3 = true; [[maybe_unused]] constexpr bool s_use_ssse3 = true;
constexpr bool s_use_sse4_1 = true; constexpr bool s_use_sse4_1 = true;
constexpr bool s_use_avx2 = true; constexpr bool s_use_avx2 = true;
constexpr bool s_use_avx3 = false; [[maybe_unused]] constexpr bool s_use_avx3 = false;
#elif defined(__SSE4_1__) #elif defined(__SSE4_1__)
constexpr bool s_use_ssse3 = true; [[maybe_unused]] constexpr bool s_use_ssse3 = true;
constexpr bool s_use_sse4_1 = true; constexpr bool s_use_sse4_1 = true;
constexpr bool s_use_avx2 = false; constexpr bool s_use_avx2 = false;
constexpr bool s_use_avx3 = false; [[maybe_unused]] constexpr bool s_use_avx3 = false;
#elif defined(__SSSE3__) #elif defined(__SSSE3__)
constexpr bool s_use_ssse3 = true; [[maybe_unused]] constexpr bool s_use_ssse3 = true;
constexpr bool s_use_sse4_1 = false; constexpr bool s_use_sse4_1 = false;
constexpr bool s_use_avx2 = false; constexpr bool s_use_avx2 = false;
constexpr bool s_use_avx3 = false; [[maybe_unused]] constexpr bool s_use_avx3 = false;
#elif defined(ARCH_X64) #elif defined(ARCH_X64)
const bool s_use_ssse3 = utils::has_ssse3(); [[maybe_unused]] const bool s_use_ssse3 = utils::has_ssse3();
const bool s_use_sse4_1 = utils::has_sse41(); const bool s_use_sse4_1 = utils::has_sse41();
const bool s_use_avx2 = utils::has_avx2(); const bool s_use_avx2 = utils::has_avx2();
const bool s_use_avx3 = utils::has_avx512(); [[maybe_unused]] const bool s_use_avx3 = utils::has_avx512();
#else #else
constexpr bool s_use_ssse3 = true; // Non x86 [[maybe_unused]] constexpr bool s_use_ssse3 = true; // Non x86
constexpr bool s_use_sse4_1 = true; // Non x86 constexpr bool s_use_sse4_1 = true; // Non x86
constexpr bool s_use_avx2 = false; constexpr bool s_use_avx2 = false;
constexpr bool s_use_avx3 = false; [[maybe_unused]] constexpr bool s_use_avx3 = false;
#endif #endif
const __m128i s_bswap_u32_mask = _mm_set_epi8( const __m128i s_bswap_u32_mask = _mm_set_epi8(

View file

@ -10,7 +10,7 @@
namespace vk namespace vk
{ {
AsyncTaskScheduler::AsyncTaskScheduler(vk_gpu_scheduler_mode mode) AsyncTaskScheduler::AsyncTaskScheduler([[maybe_unused]] vk_gpu_scheduler_mode mode)
{ {
if (g_cfg.video.renderer != video_renderer::vulkan || !g_cfg.video.vk.asynchronous_texture_streaming) if (g_cfg.video.renderer != video_renderer::vulkan || !g_cfg.video.vk.asynchronous_texture_streaming)
{ {

View file

@ -89,7 +89,7 @@ namespace vk
using download_buffer_object = void*; using download_buffer_object = void*;
using barrier_descriptor_t = rsx::deferred_clipped_region<vk::render_target*>; using barrier_descriptor_t = rsx::deferred_clipped_region<vk::render_target*>;
static std::pair<VkImageUsageFlags, VkImageCreateFlags> get_attachment_create_flags(VkFormat format, u8 /*samples*/) static std::pair<VkImageUsageFlags, VkImageCreateFlags> get_attachment_create_flags(VkFormat format, [[maybe_unused]] u8 samples)
{ {
if (g_cfg.video.strict_rendering_mode) if (g_cfg.video.strict_rendering_mode)
{ {

View file

@ -390,9 +390,8 @@ void fmt_class_string<blit_engine::transfer_operation>::format(std::string& out,
case blit_engine::transfer_operation::srccopy: return "srccopy"; case blit_engine::transfer_operation::srccopy: return "srccopy";
case blit_engine::transfer_operation::srccopy_and: return "srccopy_and"; case blit_engine::transfer_operation::srccopy_and: return "srccopy_and";
case blit_engine::transfer_operation::srccopy_premult: return "srccopy_premult"; case blit_engine::transfer_operation::srccopy_premult: return "srccopy_premult";
default: return unknown;
} }
return unknown;
}); });
} }
@ -416,9 +415,8 @@ void fmt_class_string<blit_engine::transfer_source_format>::format(std::string&
case blit_engine::transfer_source_format::x8r8g8b8: return "x8r8g8b8"; case blit_engine::transfer_source_format::x8r8g8b8: return "x8r8g8b8";
case blit_engine::transfer_source_format::y8: return "y8"; case blit_engine::transfer_source_format::y8: return "y8";
case blit_engine::transfer_source_format::yb8cr8ya8cb8: return "yb8cr8ya8cb8"; case blit_engine::transfer_source_format::yb8cr8ya8cb8: return "yb8cr8ya8cb8";
default: return unknown;
} }
return unknown;
}); });
} }
@ -447,9 +445,8 @@ void fmt_class_string<blit_engine::transfer_destination_format>::format(std::str
case blit_engine::transfer_destination_format::a8r8g8b8: return "a8r8g8b8"; case blit_engine::transfer_destination_format::a8r8g8b8: return "a8r8g8b8";
case blit_engine::transfer_destination_format::r5g6b5: return "r5g6b5"; case blit_engine::transfer_destination_format::r5g6b5: return "r5g6b5";
case blit_engine::transfer_destination_format::y32: return "y32"; case blit_engine::transfer_destination_format::y32: return "y32";
default: return unknown;
} }
return unknown;
}); });
} }

View file

@ -52,9 +52,8 @@ void fmt_class_string<QVideoFrame::PixelFormat>::format(std::string& out, u64 ar
case QVideoFrame::Format_ABGR32: return "ABGR32"; case QVideoFrame::Format_ABGR32: return "ABGR32";
case QVideoFrame::Format_YUV422P: return "YUV422P"; case QVideoFrame::Format_YUV422P: return "YUV422P";
case QVideoFrame::Format_User: return "User"; case QVideoFrame::Format_User: return "User";
default: return unknown;
} }
return unknown;
}); });
} }

View file

@ -933,7 +933,7 @@ template <typename F>
FORCE_INLINE auto root_info::slot_search(uptr iptr, u128 mask, F func) noexcept FORCE_INLINE auto root_info::slot_search(uptr iptr, u128 mask, F func) noexcept
{ {
u32 index = 0; u32 index = 0;
u32 total = 0; [[maybe_unused]] u32 total = 0;
for (hash_engine _this(iptr);; _this.advance()) for (hash_engine _this(iptr);; _this.advance())
{ {