Since the introduction of this library, numerous improvements have been
made. Notably, many of the warnings we would get by simply including the
library header have now been fixed. This makes it much easier to make
conversion warning an error.

These are fairly trivial to resolve and most of the changes entail
using RESULT_UNKNOWN over ResultCode(-1).

We simply need to turn the literal argument to std::accumulate into a
double, rather than an int.

Uncovered a bug within Thread's SetCoreAndAffinityMask() where an
unsigned variable (ideal_core) was being compared against "< 0", which
would always be a false condition.

We can also get rid of an unused function (GetNextProcessorId) which contained a sign
mismatch warning.

Resolves a few trivial sign conversion/mismatch errors.

Will be used to reduce the overall duplication of the same magic value
all over the codebase in following changes.

The constructor was implicitly using signed->unsigned conversions to
produce 0xFFFFFFFF. We can just specify this explicitly with UINT32_MAX.

While we're at it, this also resolves a type truncation warning as well,
given the code was truncating from a 64-bit value to a 32-bit one.

Quite frequently there have been cases where code has been merged into
the core that produces warning. In order to prevent this from occurring,
we can make the compiler flag these cases and allow our CI to flag down
any code that would generate these warnings.

This is beneficial given silent conversions from signed/unsigned can
result in logic bugs. This forces one writing changes to be explicit
about when signedness conversions are desirable, rather than leaving it
up to readers' interpretation.

Currently the codebase isn't in a state where it will build successfully
with this change applied, but this will be addressed in subsequent
follow-up changes. This set of changes will focus on making it build
properly with these changes for MSVC as a starting point for basic
coverage.

yuzu: configure_web: Use Base64 encoded token

gl_shader_cache: Fix locker constructors

GLSLDecompiler: Correct Texture Gather Offset.

Properly pass engine when a shader is being constructed from memory.

This commit corrects the argument ordering in textureGatherOffset.

shader/control_flow: Abstract repeated code chunks in BRX tracking

`boost::make_iterator_range` is available when `boost/range/iterator_range.hpp` is included.
Also include `boost/icl/interval_map.hpp` and `boost/icl/interval_set.hpp`.

shader_ir: Reduce severity of warnings

Update src/video_core/shader/control_flow.cpp

Co-Authored-By: Mat M. <mathew1800@gmail.com>

Update src/video_core/shader/control_flow.cpp

Co-Authored-By: Mat M. <mathew1800@gmail.com>

Update src/video_core/shader/control_flow.cpp

Co-Authored-By: Mat M. <mathew1800@gmail.com>

Update src/video_core/shader/control_flow.cpp

Co-Authored-By: Mat M. <mathew1800@gmail.com>

Update src/video_core/shader/control_flow.cpp

Co-Authored-By: Mat M. <mathew1800@gmail.com>

Update src/video_core/shader/control_flow.cpp

Co-Authored-By: Mat M. <mathew1800@gmail.com>

Remove copied and pasted for cycles into a common templated function.

These containers have a default constructor.

These warnings don't offer meaningful information while decoding
shaders. Remove them.

ci: Rename build folder only on non-mainline builds

Revert "ci: Rename build folder only on non-mainline builds"

gl_rasterizer: Upload constant buffers with glNamedBufferSubData

ci: Populate build repository from Azure environment

kernel: Improve events

shader/node: Unpack bindless texture encoding

ci: Rename build folder only on non-mainline builds