[icc] fix intel c++ compiler errors

docs/build/Linux.md

@@ -33,13 +33,12 @@ Dependencies are listed here as commands that can be copied/pasted. Of course, t
   - Proper wayland support requires `qt6-wayland`
   - GCC 11 or later is required.
 - Ubuntu / Linux Mint / Debian:
-  - `sudo apt-get install autoconf cmake g++-11 gcc-11 git glslang-tools libasound2 libboost-context-dev libglu1-mesa-dev libhidapi-dev libpulse-dev libtool libudev-dev libxcb-icccm4 libxcb-image0 libxcb-keysyms1 libxcb-render-util0 libxcb-xinerama0 libxcb-xkb1 libxext-dev libxkbcommon-x11-0 mesa-common-dev nasm ninja-build qtbase6-dev qtbase6-private-dev qtwebengine6-dev qtmultimedia6-dev libmbedtls-dev catch2 libfmt-dev liblz4-dev nlohmann-json3-dev libzstd-dev libssl-dev libavfilter-dev libavcodec-dev libswscale-dev`
+  - `sudo apt-get install autoconf cmake g++ gcc git glslang-tools libasound2 libboost-context-dev libglu1-mesa-dev libhidapi-dev libpulse-dev libtool libudev-dev libxcb-icccm4 libxcb-image0 libxcb-keysyms1 libxcb-render-util0 libxcb-xinerama0 libxcb-xkb1 libxext-dev libxkbcommon-x11-0 mesa-common-dev nasm ninja-build qtbase6-dev libmbedtls-dev catch2 libfmt-dev liblz4-dev nlohmann-json3-dev libzstd-dev libssl-dev libavfilter-dev libavcodec-dev libswscale-dev pkg-config zlib1g-dev`
   - Ubuntu 22.04, Linux Mint 20, or Debian 12 or later is required.
   - Users need to manually specify building with QT Web Engine enabled.  This is done using the parameter `-DYUZU_USE_QT_WEB_ENGINE=ON` when running CMake.
-  - Users need to manually specify building with GCC 11. This can be done by adding the parameters `-DCMAKE_C_COMPILER=gcc-11 -DCMAKE_CXX_COMPILER=g++-11` when running CMake. i.e.
   - Users need to manually disable building SDL2 from externals if they intend to use the version provided by their system by adding the parameters `-DYUZU_USE_EXTERNAL_SDL2=OFF`
 
-```
+```sh
 git submodule update --init --recursive
 cmake .. -GNinja -DCMAKE_C_COMPILER=gcc-11 -DCMAKE_CXX_COMPILER=g++-11
 ```

src/CMakeLists.txt

@@ -124,7 +124,7 @@ else()
         -Wno-missing-field-initializers
     )
 
-if (CMAKE_CXX_COMPILER_ID MATCHES Clang) # Clang or AppleClang
+    if (CMAKE_CXX_COMPILER_ID MATCHES Clang OR CMAKE_CXX_COMPILER_ID MATCHES IntelLLVM) # Clang or AppleClang
         add_compile_options(
             -Wno-braced-scalar-init
             -Wno-unused-private-field

src/audio_core/sink/sink_stream.cpp

@@ -44,7 +44,7 @@ void SinkStream::AppendBuffer(SinkBuffer& buffer, std::span<s16> samples) {
         // Center = 0.596
         // LFE = 0.354
         // Back = 0.707
-        static constexpr std::array<f32, 4> down_mix_coeff{1.0, 0.596f, 0.354f, 0.707f};
+        static constexpr std::array<f32, 4> down_mix_coeff{1.0f, 0.596f, 0.354f, 0.707f};
 
         for (u32 read_index = 0, write_index = 0; read_index < samples.size();
              read_index += system_channels, write_index += device_channels) {

src/hid_core/resources/npad/npad_vibration.cpp

@@ -69,7 +69,7 @@ Result NpadVibration::GetVibrationMasterVolume(f32& out_volume) const {
 Result NpadVibration::BeginPermitVibrationSession(u64 aruid) {
     std::scoped_lock lock{mutex};
     session_aruid = aruid;
-    volume = 1.0;
+    volume = 1.0f;
     return ResultSuccess;
 }
 

src/input_common/drivers/sdl_driver.cpp

@@ -110,12 +110,12 @@ public:
 
     bool RumblePlay(const Common::Input::VibrationStatus vibration) {
         constexpr u32 rumble_max_duration_ms = 2000;
-        constexpr f32 low_start_sensitivity_limit = 140.0;
+        constexpr f32 low_start_sensitivity_limit = 140.0f;
-        constexpr f32 low_width_sensitivity_limit = 400.0;
+        constexpr f32 low_width_sensitivity_limit = 400.0f;
-        constexpr f32 high_start_sensitivity_limit = 200.0;
+        constexpr f32 high_start_sensitivity_limit = 200.0f;
-        constexpr f32 high_width_sensitivity_limit = 700.0;
+        constexpr f32 high_width_sensitivity_limit = 700.0f;
         // Try to provide some feeling of the frequency by reducing the amplitude depending on it.
-        f32 low_frequency_scale = 1.0;
+        f32 low_frequency_scale = 1.0f;
         if (vibration.low_frequency > low_start_sensitivity_limit) {
             low_frequency_scale =
                 std::max(1.0f - (vibration.low_frequency - low_start_sensitivity_limit) /
@@ -124,7 +124,7 @@ public:
         }
         f32 low_amplitude = vibration.low_amplitude * low_frequency_scale;
 
-        f32 high_frequency_scale = 1.0;
+        f32 high_frequency_scale = 1.0f;
         if (vibration.high_frequency > high_start_sensitivity_limit) {
             high_frequency_scale =
                 std::max(1.0f - (vibration.high_frequency - high_start_sensitivity_limit) /

src/input_common/input_engine.cpp

@@ -249,7 +249,7 @@ void InputEngine::ResetButtonState() {
 void InputEngine::ResetAnalogState() {
     for (const auto& controller : controller_list) {
         for (const auto& axis : controller.second.axes) {
-            SetAxis(controller.first, axis.first, 0.0);
+            SetAxis(controller.first, axis.first, 0.0f);
         }
     }
 }

src/shader_recompiler/backend/spirv/emit_spirv_floating_point.cpp

@@ -165,8 +165,8 @@ Id EmitFPSaturate16(EmitContext& ctx, Id value) {
 }
 
 Id EmitFPSaturate32(EmitContext& ctx, Id value) {
-    const Id zero{ctx.Const(f32{0.0})};
+    const Id zero{ctx.Const(f32{0.0f})};
-    const Id one{ctx.Const(f32{1.0})};
+    const Id one{ctx.Const(f32{1.0f})};
     return Clamp(ctx, ctx.F32[1], value, zero, one);
 }
 

src/shader_recompiler/frontend/maxwell/decode.cpp

@@ -51,7 +51,8 @@ struct InstEncoding {
     MaskValue mask_value;
     Opcode opcode;
 };
-constexpr std::array UNORDERED_ENCODINGS{
+constexpr auto SortedEncodings() {
+    std::array<InstEncoding, 279> encodings{
 #define INST(name, cute, encode)                                                                   \
     InstEncoding{                                                                                  \
         .mask_value{MaskValueFromEncoding(encode)},                                                \
@@ -60,9 +61,6 @@ constexpr std::array UNORDERED_ENCODINGS{
 #include "maxwell.inc"
 #undef INST
     };
-
-constexpr auto SortedEncodings() {
-    std::array encodings{UNORDERED_ENCODINGS};
     std::ranges::sort(encodings, [](const InstEncoding& lhs, const InstEncoding& rhs) {
         return std::popcount(lhs.mask_value.mask) > std::popcount(rhs.mask_value.mask);
     });

src/shader_recompiler/frontend/maxwell/opcodes.cpp

@@ -2,24 +2,22 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 
 #include <array>
+#include <string_view>
 
 #include "shader_recompiler/exception.h"
 #include "shader_recompiler/frontend/maxwell/opcodes.h"
 
 namespace Shader::Maxwell {
-namespace {
+
-constexpr std::array NAME_TABLE{
+const char* NameOf(Opcode opcode) {
+    constexpr const char* NAME_TABLE[] = {
 #define INST(name, cute, encode) cute,
 #include "maxwell.inc"
 #undef INST
     };
-} // Anonymous namespace
+    if (size_t(opcode) >= sizeof(NAME_TABLE) / sizeof(NAME_TABLE[0]))
-
+        throw InvalidArgument("Invalid opcode with raw value {}", int(opcode));
-const char* NameOf(Opcode opcode) {
+    return NAME_TABLE[size_t(opcode)];
-    if (static_cast<size_t>(opcode) >= NAME_TABLE.size()) {
-        throw InvalidArgument("Invalid opcode with raw value {}", static_cast<int>(opcode));
-    }
-    return NAME_TABLE[static_cast<size_t>(opcode)];
 }
 
 } // namespace Shader::Maxwell