video_core/engines: Remove unnecessary inclusions where applicable

video_core/memory_manager: Mark a few member functions with the const qualifier

file_sys/fsmitm_romfsbuild: Utilize a string_view in romfs_calc_path_hash

core: Add missing override specifiers where applicable

video_core/gpu_thread: Silence truncation warning in ThreadManager's constructor

file_sys/nca_metadata: Remove unnecessary comparison operators for TitleType

service/fsp_srv: Update SaveDataInfo and SaveDataDescriptor structs

gl_shader_decompiler: Return early when an operation is invalid

file_sys/program_metadata: Remove obsolete TODOs

gl_shader_decompiler: Rename GenerateTemporal() to GenerateTemporary()

kernel/svc: Properly sanitize mutex address in WaitProcessWideKeyAtomic

hle/result: Remove unnecessary bitfield entry for ResultCode

yuzu/debugger/graphics/graphics_surface: General cleanup

video_core/macro_interpreter: Simplify GetRegister()

We can just use .at(), which essentially does the same thing, but with
less code.

Given we already ensure nothing can set the zeroth register in
SetRegister(), we don't need to check if the index is zero and special
case it. We can just access the register normally, since it's already
going to be zero.

We can also replace the assertion with .at() to perform the equivalent
behavior inline as part of the API.

Given this doesn't actually alter internal state, this can be made a
const member function.

Now, since we have a const qualified variant of GetPointer(), we can put
it to use in ReadBlock() to retrieve the source pointer that is passed
into memcpy.

Now block reading may be done from a const context.

Allows retrieving read-only pointers from a const context externally.

This doesn't modify internal state, so it can be made a const member
function.

This doesn't modify any internal state, so it can be made a const member
function to allow its use in const contexts.

- Use QStringLiteral where applicable.

- Use const where applicable

- Remove unnecessary precondition check (we already assert the pixbuf
  being non null)

We can utilize qOverload with the signal connections to make the
function deducing a little less ugly.

Fills in the missing surface types that were marked as unknown. The
order corresponds with the TextureFormat enum within
video_core/texture.h.

We also don't need to all of these strings as translatable (only the
first string, as it's an English word).

Since c5d41fd8 callback parameters were
changed to use an s64 to represent late cycles instead of an int, so
this was causing a truncation warning to occur here. Changing it to s64
is sufficient to silence the warning.

These aren't used externally by anything, so they can be made private
data members.

Replaces header inclusions with forward declarations where applicable
and also removes unused headers within the cpp file. This reduces a few
more dependencies on core/memory.h

This is a hold over from the 3DS error codes in Citra.

BitField has been trivially copyable since
e99a1486, so we can eliminate these
TODO comments and use ReadObject() directly instead of memcpying the
data.

enum class elements from the same enum can already be compared against
one another without the need for explicitly defined comparison
operators.

The given string instance doesn't need to be copied entirely, we can
just use a view instead.

Passing around a 64 byte data struct by value is kind of wasteful,
instead pass a reference to the struct.

The unknown member here is actually padding due to being passed as a
struct. We can do the same, and remove the need to pop a padding word.

I realized that I updated the documentation on SwitchBrew a while ago,
but never actually updated the structs within yuzu.

filesystem: Use a std::string_view in OpenFile()

gpu_thread: Improve synchronization by using CoreTiming.

file_sys/control_metadata: Amend naming of members

hle/service: Resolve unused variable warnings

gl_shader_decompiler: Fix TXQ types