gl_device: Mark stage_swizzle as constexpr

surface_base: Make IsInside() a const member function

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

This doesn't modify internal state, so this can be made const.

video_core: Amend doxygen comment references

Eliminates redundant atomic reference count increments and decrements.

Fixes broken documentation references.

Previously this was mutable even though it shouldn't be.

Texture Cache: Read current data when flushing a 3D segment.

shader/memory: Implement RED.E.ADD and minor changes to ATOM

CMakeLists: Make -Wreorder a compile-time error

This can result in silent logic bugs within code, and given the amount
of times these kind of warnings are caused, they should be flagged at
compile-time so no new code is submitted with them.

gl_rasterizer: Implement constant vertex attributes

This PR corrects flushing of 3D segments when data of other segments is
mixed, this aims to preserve the data in place.

maxwell_to_vk: Add uint16 vertex formats

gl_shader_cache: Use CompileDepth::FullDecompile on GLSL

gl_texture_cache: Fix layered texture attachment base level

shader/arithmetic: Add FCMP_CR variant

Revert "gl_shader_decompiler: Implement merges with bitfieldInsert"

vk_blit_screen: Initialize all members in VkPipelineViewportStateCreateInfo

Avoid invalid fallbacks.

When the dynamic state is specified, pViewports and pScissors are
ignored, quoting the specification:

  pViewports is a pointer to an array of VkViewport structures, defining
  the viewport transforms. If the viewport state is dynamic, this member
  is ignored.

That said, AMD's proprietary driver itself seem to read it regardless of
what the specification says.

vk_rasterizer: Default to 1 viewports with a size of 0

This is a simple optimization as Buffer Copies are mostly used for texture recycling. They are, however, useful when games abuse undefined behavior but most 3D APIs forbid it.

This reverts commit 05cf2708.

Apparently the first approach using floats instead of bitfieldInert
worked better for Fire Emblem: Three Houses. Reverting to get that
behavior back.

Adds another variant of FCMP.

service: friend: Stub IFriendService::GetBlockedUserListIds.

file_sys: patch_manager: Return early when there are no layers to apply.

Credits go to gdkchan from Ryujinx for finding constant attributes are
used in retail games.

- This is safe to stub, as there should be no adverse consequences from reporting no blocked users.

Silence validation layer errors.

From my testing on a Splatoon 2 shader that takes 3800ms on average to
compile changing to FullDecompile reduces it to 900ms on average.

The shader decoder will automatically fallback to a more naive method if
it can't use full decompile.

The base level is already included in the texture view. If we specify
the base level in the texture again, this will end up in the incorrect
level and potentially out of bounds.

renderer_vulkan: Drop Vulkan-Hpp

gl_rasterizer: Implement line widths and smooth lines

texture_cache: Remove preserve_contents

texture/astc: More small ASTC optimizations

shader/conversion: Implement I2I sign extension, saturation and selection