Allows building on clang to work again

core/settings: Move configuring_global behind an API

General: Resolve a few missing initializer warnings

Rather than have directly modified global state here, we can make it an
implementation detail and have an interface that changes are queried
through.

Improvements to GPU synchronization & various refactoring

nvdec: Make use of [[nodiscard]] where applicable

microprofile: Silence warning in headers

Silences a truncation warning by making the truncation explicit and
documenting the reason for it.

common: Enable warnings as errors

hle: service: ldr: Implement UnloadNrr.

Cleans up common so that we can enable warnings as errors.

Prevents bugs from occurring where the results of a function are
accidentally discarded

async_shaders: Increase Async worker thread count for >8 thread cpus

- Refactor so that SubmitGPFIFO and KickoffPB use shared functionality.
- Implement add_wait and add_increment flags.

- Log corrupted command lists, rather than crash.

* Rename to align with switchbrew

* Rename to align with switchbrew and remove gpu function that checks if clearing should be done.

shader/arithmetic: Implement FCMP immediate + register variant

- Used by Final Fantasy X/X-2 HD Remaster.

video_core: unbreak -Werror in NVDEC with Clang

Prevents the compiler tripping up about Windows headers.

src/core/hle/service/nvdrv/devices/nvhost_nvdec_common.cpp:41:15: error: unused variable 'OutOfMemory' [-Werror,-Wunused-const-variable]
constexpr u32 OutOfMemory{static_cast<u32>(-12)};
              ^

General: Make ignoring a discarded return value an error

VP9: Minor interface changes and safety improvements

MSVC lets us fine-tune catching expressions with no side-effects a
little more.

Allows our CI to catch more potential bugs. This also removes the
[[nodiscard]] attribute of IOFile's Open member function. There are
cases where a file may want to be opened, but have the status of it
checked at a later time.

It's deprecated in the language to autogenerate these if the destructor
for a type is specified, so we can explicitly specify how we want these
to be generated.

Prevents values from mistakenly being discarded in cases where it's a
bug to do so.

The API of VP9 exposes a WasFrameHidden() function which accesses this
member. Given the constructor previously didn't initialize this member,
it's a potential vector for an uninitialized read.

Instead, we can initialize this to a deterministic value to prevent that
from occurring.

These helper functions don't directly modify any member state and can be
hidden from view.

Resolves a few -Wmissing-initializer warnings.

externals: Update fmt to 7.1.0