Revert #2466

This reverts a tested behavior on delay slots not exiting if the exit 
flag is set. Currently new tests are required in order to ensure this 
behavior.

shader: Implement AL2P and ALD.PHYS

 kernel/svc: Reorganize and fix up the initial handling of svcSetThreadCoreMask()

Dma_pusher: ASSERT on empty command_list

Correct possible error on Rasterizer Caches

shader/shader_ir: Minor changes

gl_shader_disk_cache: Minor cleanup

This is a measure to avoid crashes on command list reading as an empty
command_list is considered a NOP.

service/audren_u: Get rid of magic values within GetAudioRendererWorkBufferSize

video_core/gpu_thread: Remove redundant copy constructor for CommandDataContainer

service/set: Correct and simplify behavior related to copying language codes

GPU/MMEInterpreter: Ignore the 'exit' flag when it's executed inside a delay slot.

yuzu: Remove explicit types from locks where applicable

maxwell_3d: reduce severity of different component formats assert.

video_core/engines/maxwell_3d: Add is_trivially_copyable_v check for Regs

video_core/engines/maxwell_3d: Simplify for loops into ranged for loops within InitializeRegisterDefaults()

service/am: Add missing return in error case for IStorageAccessor's Read/Write()

gl_rasterizer: Pass the right number of array quad vertices count

gl_rasterizer: Limit OpenGL point size to a minimum of 1

 video_core/engines/engine_upload: Minor tidying

maxwell_to_gl: Add TriangleFan primitive topology

ipc_helpers: Amend floating-point type in Pop<double> specialization

 yuzu/debugger: Specify string conversions explicitly 

core/kernel/object: Rename ResetType enum members for clarity

kernel/svc: Mark GetThreadList() and UnmapProcessCodeMemory() as internally linked

yuzu/applets/profile_select: Mark header string as translatable

gl_shader_gen: std::move objects where applicable

CMakeLists: Handle VS 2019 in a less annoying manner

yuzu/bootmanager: Explicitly enable deprecated OpenGL features on compat

yuzu/util: Remove unused spinbox.cpp/.h

constexpr internally links by default, so the inline specifier is
unnecessary.

Many of these constructors don't even need to be templated. The only
ones that need to be templated are the ones that actually make use of
the parameter pack.

Even then, since std::vector accepts an initializer list, we can supply
the parameter pack directly to it instead of creating our own copy of
the list, then copying it again into the std::vector.

shader/shader_ir: Remove unnecessary template parameter packs from Operation() overloads where applicable

These overloads don't actually make use of the parameter pack, so they
can be turned into regular non-template function overloads.

These don't actually modify instance state, so they can be marked as
const member functions

This has been left unused since the removal of the vestigial surface
viewer. Given it has no uses left, this can be removed as well.

Given the class contains quite a lot of non-trivial types, place the
constructor and destructor within the cpp file to avoid inlining
construction and destruction code everywhere the class is used.

Avoids performing copies into the pair being returned. Instead, we can
just move the resources into the pair, avoiding the need to make copies
of both the std::string and ShaderEntries struct.

Given the offset is assigned a fixed value in the constructor, we can
just assign it directly and get rid of the need to write the name of the
variable again in the constructor initializer list.

Given the disk shader cache contains non-trivial types, we should
default it in the cpp file in order to prevent inlining of the
complex destruction logic.