Given we just recently had a patch backport this from citra, let's try
and keep the convention uniform.

Given these are only added to the class to allow those functions to
access the private constructor, it's a better approach to just make them
static functions in the interface, to make the dependency explicit.

This converts it into a regular constructor parameter. There's no need
to make this a template parameter on the class when it functions
perfectly well as a constructor argument.

This also reduces the amount of code bloat produced by the compiler, as
it doesn't need to generate the same code for multiple different
instantiations of the same class type, but with a different fill value.

* Implemented fatal:u properly

fatal:u now is properly implemented with all the ipc cmds. Error reports/Crash reports are also now implemented for fatal:u. Crash reports save to yuzu/logs/crash_reports/
The register dump is currently known as sysmodules send all zeros. If there are any non zero values for the "registers" or the unknown values, let me know!

* Fatal:U fixups

* Made fatal:u execution break more clear

* Fatal fixups

* Stubbed IRS

Currently we have no ideal way of implementing IRS. For the time being we should have the functions stubbed until we come up with a way to emulate IRS properly.

* Added IRS to logging backend

* Forward declared shared memory for irs

* Added glObjectLabels for renderdoc for textures and shader programs

* Changed hardcoded "Texture" name to reflect the texture type instead

* Removed string initialize

simply use the standard library yield()

Reads multiple dirs through as if a waterfall.

Maps a vector into the VFS interface.

Always returns the template argument byte for all reads. Doesn't support writes.

Maps name string to directory or file.

This isn't used anywhere within the header, so we can remove it, along
with the include that was previously necessary. This also uncovers an
indirect include in the cpp file for the assertion macros.

This is more consistent with the surrounding enums.

These don't affect the state of the struct and can be const member
functions.

This was very likely intended to be a logical OR based off the
conditioning and testing of inversion in one case.

Even if this was intentional, this is the kind of non-obvious thing one
should be clarifying with a comment.

Reduces the use of Process class members externally and keeps most code
related to tearing down a process with the rest of the process code.

Allows making several members of the process class private, it also
avoids going through Core::CurrentProcess() just to retrieve the owning
process.

* Added support for uncompressed NSOs

* Moved compressed section check to NsoHeader

This function is called too many times and makes the debug logging basically unusable due to the spam.

Previously we were clearing the default backbuffer framebuffer.

Found thanks to a Piglit test :)

Even though setting this value to 3 is more correct. We break more games than we fix due to missing implementations. We should keep this as 0 for the time being

The owning process of a thread is required to exist before the thread,
so we can enforce this API-wise by using a reference. We can also avoid
the reliance on the system instance by using that parameter to access
the page table that needs to be set.

Avoids an unnecessary inclusion and also uncovers three places where
indirect inclusions were relied upon, which allows us to also resolve
those.