service/nvflinger: Minor renaming changes

Add missing environment variables to travis-ci.env

video_core/dma_pusher: Silence C4828 warnings

This was previously causing:

warning C4828: The file contains a character starting at offset 0xa33
that is illegal in the current source character set (codepage 65001).

warnings on Windows when compiling yuzu.

Add more info into textures' object labels

This member function doesn't actually modify instance state, so it can
be const-qualified.

This more accurately describes what the function is actually attempting
to do (it's not a simple trivial getter).

gl_shader_cache: Fix texture view for cubemaps as cubemap arrays

service/nvflinger: Add the null display type

gl_shader_cache: Use explicit bindings

video_core: Implement LDG through heuristics based on IR

* Implemented the puller semaphore operations.

* Nit: Fix 2 style issues

* Nit: Add Break to default case.

* Fix style.

* Update for comments. Added ReferenceCount method

* Forgot to remove GpuSmaphoreAddress union.

* Fix the clang-format issues.

* More clang formatting.

* two more white spaces for the Clang formatting.

* Move puller members into the regs union

* Updated to use Memory::WriteBlock instead of Memory::Write*

* Fix clang style issues

* White space clang error

* Removing unused funcitons and other pr comment

* Removing unused funcitons and other pr comment

* More union magic for setting regs value.

* union magic refcnt as well

*  Remove local var

* Set up the regs and regs_assert_positions up properly

* Fix clang error

Cubemaps are considered layered and to create a texture view the texture
mustn't be a layered texture, resulting in cubemaps being bound as
cubemap arrays. To fix this issue this commit introduces an extra
surface parameter called "is_array" and uses this to query for texture
view creation.

Now that texture views for cubemaps are actually being created, this
also fixes the number of layers created for the texture view (since they
have to be 6 to create a texture view of cubemaps).

In addition to the default, external, EDID, and internal displays,
there's also a null display provided as well, which as the name
suggests, does nothing but discard all commands given to it. This is
provided for completeness.

Opening a display isn't really a thing to warn about. It's an expected
thing, so this can be a debug log. This also alters the string to
indicate the display name better.

Opening "Default" display reads a little nicer compared to Opening
display Default.

Makes non-mutable state more explicit.

The built-in set of displays is fixed, so we can utilize an array
instead of a vector here.

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.