content_archive: Minor reorganization changes

Used better names for mm:u and fixed a bad stub

core: Remove unnecessary assert in ArmInterface()

CpuCore already does this sort of checking, so we can just call that
instead of duplicating the assertions.

XCI: Add function for checking the existence of the program NCA

Implemented 3D Textures

InitializeWithId needs to return an id which is a u32 which should be a non zero value

"Better Hid" Rework Part 1

shader_decompiler: Fix TLDS

Implement flushing in the rasterizer cache

svc: Clarify enum values for AddressSpaceBaseAddr and AddressSpaceSize in svcGetInfo()

Implement Arrays on Tex Instruction

savedata_factory: Add DeviceSaveData and fix TemporaryStorage

Reason for the change is to allow both docked and undocked mode to work

- This will be used as a catch-all for slow-but-accurate GPU emulation paths.

std::move doesn't actually dereference the data, so it doesn't matter
whether or not the type is null.

This is only ever used within the cpp file, so it can just be an
internal function.

This is the same as using std::any_of with an inverted predicate.

The constructor alone is pretty large, the reading code should be split
into its consistuent parts to make it easier to understand it without
having to build a mental model of a 300+ line function.

Each header is 512 bytes in size, which is kind of an excessive amount
to copy all the time when it's possible to avoid doing so.

The only reason the getter existed was to check whether or not the
program NCA was null. Instead, we can just provide a function to query
for the existence of it, instead of exposing it entirely.

- This does not really have anything to do with OpenGL.

- These cannot be used within the cache, as they change cache state.