Refactor GLES routine cache keys to use Memset<T>
This ports the Vulkan side change to OpenGL ES and silences the GCC 8
class-memaccess warning/error, while ensuring proper key initialization.
Defines a Memset<T> class to be used as the first base class of cache
key types, which makes it explicit that their underlying memory will be
fully initialized before any member constructors are run.
In particular this fixes Blitter::Options state having uninitialized
bits after the bitfield, and Blitter::State having uninitialized padding
bytes after Options so 'sourceFormat' is 32-bit aligned.
Also adds is_memcmparable<T> for checking if memcmp() can be used to
implement operator==() for cache keys. It's equivalent to
std::is_trivially_copyable except it provides a fallback for STL
implementations that don't support it.
Also fix class-memset violations in LLVM 7.0 with their solution from
a later version.
Bug: b/135744933
Change-Id: Ic1e5c2c6b944a5133fc55840c0508bc2cdd1d66a
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/33488
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Chris Forbes <chrisforbes@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
diff --git a/src/Renderer/PixelProcessor.hpp b/src/Renderer/PixelProcessor.hpp
index 4fa627c..b901ffe 100644
--- a/src/Renderer/PixelProcessor.hpp
+++ b/src/Renderer/PixelProcessor.hpp
@@ -28,9 +28,11 @@
class PixelProcessor
{
public:
- struct States
+ struct States : Memset<States>
{
- unsigned int computeHash();
+ States() : Memset(this, 0) {}
+
+ uint32_t computeHash();
int shaderID;
@@ -113,8 +115,6 @@
struct State : States
{
- State();
-
bool operator==(const State &state) const;
int colorWriteActive(int index) const
@@ -132,7 +132,7 @@
return pixelFogMode != FOG_NONE;
}
- unsigned int hash;
+ uint32_t hash;
};
struct Stencil