Some games (like Xenoblade Chronicles 2) clear both depth and stencil
buffers while there's a depth-only texture attached (e.g. D16 Unorm).
This commit reads the zeta format of the bound surface on
ConfigureFramebuffers and returns if depth and/or stencil attachments
were set. This is ignored on DrawArrays but on Clear it's used to just
clear those attachments, bypassing an OpenGL error.

This is kind of a large hole in the API, given we allow popping signed
values. This fixes that.

video_core/shader: Minor changes

service/pm: Implement SetMaintenanceBoot

This quite literally functions as a basic setter. No other error
checking or anything (since there's nothing to really check against).

With this, it completes the pm:bm interface in terms of functionality.

Just minor tidying of interfaces.

service/vi: Remove stubbed notifier from SetLayerVisibility

This appears to be a vestigial API function that's only kept around for
compatibility's sake, given the function only returns a success error
code and exits.

Since that's the case, we can remove the stubbed notification from the
log, since doing nothing is technically the correct behavior in this
case.

Given we're in the area, these are three trivial typos that can be
corrected.

Orders the class members in the same order that they would actually be
initialized in. Gets rid of two compiler warnings.

std::moveing a local variable in a return statement has the potential to
prevent copy elision from occurring, so this can just be converted into
a regular return.

kernel/svc: Log out uncaught C++ exceptions from svcBreak

video_core: Silent implicit conversion warning

Port citra-emu/citra#4601: "dsp_interface: fix sound being played while volume is 0"

Looking into the implementation of the C++ standard facilities that seem
to be within all modules, it appears that they use 7 as a break reason
to indicate an uncaught C++ exception.

This was primarily found via the third last function called within
Horizon's equivalent of libcxxabi's demangling_terminate_handler(),
which passes the value 0x80000007 to svcBreak.

According to documentation, if the argument of std::exp is zero, one is returned.
However we want the return value to be also zero in this case so no audio is played.

video_core: Replace gl_shader_decompiler with an IR based decompiler

frontend: Refactor ScopeAcquireWindowContext out of renderer_opengl.

Port citra-emu/citra#3928: "citra_qt: Log settings on launch"

ISSUE_TEMPLATE: changes to make it more expressive and prevent low-quality issues

maxwell_3d: Set rt_separate_frag_data to 1 by default

Commercial games assume that this value is 1 but they never set it. On
the other hand nouveau manually sets this register. On
ConfigureFramebuffers we were asserting for what we are actually
implementing (according to envytools).

yuzu/configuration: Remove unnecessary inclusions where applicable

Loading progress bar upgrades