- Used by Captain Toad: Treasure Tracker Update 1.3.0

While we're at it, fix the input parameters for SetIsPalmaAllConnectable and SetPalmaBoostMode

vk_update_descriptor: Upload descriptor sets data directly

video_core: Eliminate some variable shadowing

vulkan/wrapper: Remove noexcept from GetSurfaceCapabilitiesKHR()

Port citra-emu/citra#5322: "Fix: fatal error CVT1100 when compiling manifest file"

gl_arb_decompiler: Avoid several string copies

Occurs when doing a local compile in MSVC build. The compiler I'm using is as below:
Microsoft Visual Studio Community 2019 Preview
Version 16.6.0 Preview 5.0

Fixes this error:
CVTRES : fatal error CVT1100: duplicate resource. type:MANIFEST, name:1, language:0x0409
LINK : fatal error LNK1123: failure during conversion to COFF: file invalid or corrupt

I have put 0 since previous name was 1. If have other names in mind, please let me know.

Co-Authored-By: dragios <dragios@users.noreply.github.com>

macro_jit_x64: Use ecx for shift register

shl/shr only accept cl as their second argument

macro_jit_x64: Amend readability of Compile_ExtractShiftLeftRegister()

macro_jit_x64: Remove unused variable

Fix compilation on macOS

Variables that are marked as const cannot have the move constructor
invoked when returning from a function (the move constructor requires a
non-const variable so it can "steal" the resources from it.

Check() can throw an exception if the Vulkan result isn't successful.

We remove the check so that std::terminate isn't outright called and
allows for better debugging (should it ever actually fail).

Previously dst wasn't being used.

Previously dst wasn't being used.

Removes a completely unused label and marks another variable as unused,
given it seems like it has potential uses in the future.

Renames some variables to prevent ones in inner scopes from shadowing
outer-scoped variables.

The Copy* functions have no shadowing, but we rename them anyways to
remain consistent with the other functions.

Fix compilation when not building with boxcat

We can reduce the capture scope so that it's not possible for both "reg"
variables to clash with one another.

While we're at it, we can prevent unnecessary copies while we're at it.

We can just make use of the instance in the scope above this one.

macro_jit_x64: Inline Engines::Maxwell3D::GetRegisterValue

Remove redundant moves

Named return value optimization automatically applies here.

Named return value optimization automatically applies here.

Named return value optimization automatically applies here.

Fixes compilation when trying to build without boxcat enabled

audren: Implement RendererInfo

macro_jit_x64: Optimization correctness

maxwell_to_gl/vk: Miscellaneous changes

Instead of copying to a temporary payload before sending the update task
to the worker thread, insert elements to the payload directly.

arm_dynarmic_32: Fix implicit conversion error in SetTPIDR_EL0

On MSVC builds we treat conversion warnings as errors.

OpenGL macOS headers definitions clash heavily with each other