From what I understand, this tells CMake to use the system, not conan,
version of zstd. Required to build on the coming MinGW Docker container.

audio_core: Resolve sign conversion warnings 

vk_stream_buffer: Fix initializing Vulkan with NVIDIA on Linux

shader/registry: Make use of designated initializers where applicable 

vk_command_pool: Add missing header guard

audio_core: Remove unnecessary inclusions

behavior_info: Fix typo Renerer -> Renderer

The previous fix only partially solved the issue, as only certain GPUs that needed 9 or less MiB subtracted would work (i.e. GTX 980 Ti, GT 730). This takes from DXVK's example to divide `heap_size` by 2 to determine `allocable_size`. Additionally tested on my Quadro K4200, which previously required setting it to 12 to boot.

Same behavior, but removes header dependencies where they don't need to
be.

While were at it, we can also enable sign conversion warnings and other
common warnings as errors to prevent these from creeping back into the
codebase.

While we're at it we can make the destructor of the base class virtual
to ensure that any polymorphism issues never occur.

Allows the implementation details to be changed without recompiling any
files that include this header.

service: Restore "unused" function

Turns out this function is actually used, but within a trace log.

arithmetic_integer_immediate: Make use of std::move where applicable

acc: Partially implement LoadOpenContext

Same behavior, minus any redundant atomic reference count increments and
decrements.

renderer_vulkan: Make unconditional use of VK_KHR_timeline_semaphore

Add automap feature for GC adapter

Using statements already make these unnecessary.

memory: Resolve a -Wdocumentation warning

install_dialog: Remove unused function prototype

game_list: Eliminate redundant argument copies 

Same behavior, less repetition.

memory doesn't exist as a parameter any more.

Allows the compiler to warn against cases where the return value isn't
used (which would be a bug).

This function doesn't have an implementation, so it can be removed to
prevent others from unintentionally using it.

Makes the naming consistent with the rest of the functions that are
present.

Several functions can be taken by const reference to avoid copies

control_flow: Make use of std::move in InsertBranch()

Places data structures where they'll eventually be moved to to avoid
needing to even move them in the first place.

Avoids unnecessary atomic increments and decrements.

General: Make use of std::nullopt where applicable

Allows some implementations to avoid completely zeroing out the internal
buffer of the optional, and instead only set the validity byte within
the structure.

This also makes it consistent how we return empty optionals.

ips_layer: Eliminate a redundant copy in Parse()