| Age | Commit message (Collapse) | Author |
|---|
|
Some users who get this error don't seem to gain much understanding
of how to resolve the problem from reading the error message.
|
|
These are only ever used with ShaderCode instances and nothing else.
Given that, we can convert these helper functions to expect that type of
object as an argument and remove the need for templates, improving
compiler throughput a marginal amount, as the template instantiation
process doesn't need to be performed.
We can also move the definitions of these functions into the cpp file,
which allows us to remove a few inclusions from the ShaderGenCommon
header. This uncovered a few instances of indirect inclusions being
relied upon in other source files.
One other benefit is this allows changes to be made to the definitions
of the functions without needing to recompile all translation units that
make use of these functions, making change testing a little quicker.
Moving the definitions into the cpp file also allows us to completely
hide DefineOutputMember() from external view, given it's only ever used
inside of GenerateVSOutputMembers().
|
|
glReadPixels() with depth formats is not supported.
Should fix broken EFB access on GLES.
|
|
|
|
copies.
Consolidate the NVIDIA Tegra and non-Tegra checks into one branch.
|
|
|
|
Now that we've extracted all of the stateless functions that can be
hidden, it's time to make the index generator a regular class with
active data members.
This can just be a member that sits within the vertex manager base
class. By deglobalizing the state of the index generator we also get rid
of the wonky dual-initializing that was going on within the OpenGL
backend.
Since the renderer is always initialized before the vertex manager, we
now only call Init() once throughout the execution lifecycle.
|
|
|
|
Previously the logging was a in a little bit of a disarray. Some things
were in namespaces, and other things were not.
Given this code will feature a bit of restructuring during the
transition over to fmt, this is a good time to unify it under a single
namespace and also remove functions and types from the global namespace.
Now, all functions and types are under the Common::Log namespace. The
only outliers being, of course, the preprocessor macros.
|
|
Same thing, but nicer to read from left to right.
|
|
Same thing, less reading.
|
|
Removes a few inclusions that aren't necessary, reducing the number of
header dependencies.
|
|
|
|
This header doesn't actually make use of MathUtil.h within itself, so
this can be removed. Many other source files used VideoCommon.h as an
indirect include to include MathUtil.h, so these includes can also be
adjusted.
While we're at it, we can also migrate valid inclusions of VideoCommon.h
into cpp files where it can feasibly be done to minimize propagating it
via other headers.
|
|
Makes the global variable follow our convention of prefixing g_ on
global variables to make it obvious in surrounding code that it's not a
local variable.
|
|
Normalizes all variables related to statistics so that they follow our
coding style.
These are relatively low traffic areas, so this modification isn't too
noisy.
|
|
Renderer: Adjust source rectangle when crop would draw off screen
|
|
Since C++17, non-member std::size() is present in the standard library
which also operates on regular C arrays. Given that, we can just replace
usages of ArraySize with that where applicable.
In many cases, we can just change the actual C array ArraySize() was
called on into a std::array and just use its .size() member function
instead.
In some other cases, we can collapse the loops they were used in, into a
ranged-for loop, eliminating the need for en explicit bounds query.
|
|
|
|
Greatly simplifies the overall interface when it comes to compiling
shaders. Also allows getting rid of a std::string overload of the same
name. Now std::string and const char* both go through the same function.
|
|
|
|
Remove redundant initialization
|
|
|
|
|
|
This prevents us from requiring an oversized and/or negative viewport by
shrinking the source rectangle instead.
|
|
Has a better chance of avoiding recompiling if so.
|
|
It is always enabled in GLES.
|
|
This was occurring if the imgui vertex format was bound on shutdown,
which is destroyed before the vertex buffers
|
|
So it can be dumped with info log when linking fails.
|
|
|
|
|
|
|
|
"Cache data" can be used to assist a driver with creating pipelines by
using previously-compiled shader ISA.
|
|
Qt/HacksWidget: Fix backend feature support checks
|
|
|
|
D3D: Move sharable D3D11/D3D12 code to common library
|
|
|
|
ZTP was uploading 10MB+ of uniforms per frame, reducing paralellism by
forcing GPU waits.
|
|
Prevents attempting to create invalid MSAA configurations, for example.
|
|
|
|
|
|
D3D11 doesn't support partial copies of depth buffers via
CopySubResource(), so we need to use a different path for the EFB cache.
|
|
Was using UNORM instead of UINT, resulting in invalid values.
|
|
|
|
Use empty instead of size
|
|
|
|
Stereoscopy regression fixes
|
|
|
|
|
|
This is a scaling factor, used for hi-dpi configurations.
|
