kernel/address_arbiter: Convert the address arbiter into a class

hle/service/audio/audout_u: Correct lack of return in failure case of AppendAudioOutBufferImpl()

yuzu/debugger/wait_tree: Remove use of global CurrentProcess accessor

Previously we were overwriting the error case with a success code
further down (which is definitely not what we should be doing here).

svc: Move memory range checking functions to the VMManager class

hle/service/audio: Extract audio error codes to a header

video_core/engines: Remove unnecessary includes

{kernel/thread, video_core/surface}: Remove obsolete TODOs

We already have the thread instance that was created under the current
process, so we can just pass the handle table of it along to retrieve
the owner of the mutex.

Removes a few unnecessary dependencies on core-related machinery, such
as the core.h and memory.h, which reduces the amount of rebuilding
necessary if those files change.

This also uncovered some indirect dependencies within other source
files. This also fixes those.

Places all error codes in an easily includable header.

This also corrects the unsupported error code (I accidentally used the
hex value when I meant to use the decimal one).

Allows getting rid of reliance on the global accessor functions and
instead operating on the provided system instance.

- Invert conditions into guard clases where applicable.
- Mark std::vector parameter of WakeThreads as const

Places all of the functions for address arbiter operation into a class.
This will be necessary for future deglobalizing efforts related to both
the memory and system itself.

This isn't needed anymore, according to Hexagon

This is a TODO carried over from Citra that doesn't apply here.

Port citra-emu/citra#4630: "Memory: don't lock hle mutex in memory read/write"

Significantly tidies up two guard conditionals.

Makes the interface uniform when it comes to checking various memory
regions.

Provides a bit of a more proper interface for these functions.

 gl_rasterizer: Unbind textures but don't apply the gl_state 

 logging/backend: Move CreateEntry into the Impl class. Relocate local static to a class variable

web_service: Remove unnecessary inclusions

Reduces the potential amount of rebuilding necessary if any headers
change. In particular, we were including a header from the core library
when we don't even link the core library to the web_service library, so
this also gets rid of an indirect dependency.

Port citra-emu/citra#4647: "citra_qt/main: make SPEED_LIMIT_STEP static constexpr"

Moves local global state into the Impl class itself and initializes it
at the creation of the instance instead of in the function.

This makes it nicer for weakly-ordered architectures, given the
CreateEntry() class won't need to have atomic loads executed for each
individual call to the CreateEntry class.

This function is only ever used within this source file and makes it
easier to remove static state in the following change.

vk_buffer_cache: Fix clang-format

Yuzu can render 3D.

MSVC does not seem to like using constexpr values in a lambda that were declared outside of it.
Previously on MSVC build the hotkeys to inc-/decrease the speed limit were not working correctly because in the lambda the SPEED_LIMIT_STEP had garbage values.
After googling around a bit I found: https://github.com/codeplaysoftware/computecpp-sdk/issues/95 which seems to be a similar issue.
Trying the suggested fix to make the variable static constexpr also fixes the bug here.

Yuzu can now render 3D graphics to some degree.