Fix pointer error from real bool type

Having replaced deBool with real bool it is no longer guaranteed to be
the size of an int. This means that we need to be careful when copying
from a shader buffer where bools are 4 bytes.

In this example, from findLSBZero, 'bool comparison' was a single byte,
placed before 'int32_t res', meaning that the first 3 bytes of 'res'
were overwritten by 'comparison'.

Components: OpenGLES
Affects: dEQP-GLES31.functional.shaders.builtin_functions.uniform.*

Change-Id: I6c9c0512f97598ac4a3bca5164e1e6400ec7c321

modules/gles31/functional/es31fShaderUniformIntegerFunctionTests.cpp

@@ -98,10 +98,8 @@ void UniformIntegerFunctionCase::deinit(void)
 void UniformIntegerFunctionCase::init(void)
 {
     std::ostringstream oss;
-    oss << "result = " << getFunctionName()
-        << "(value);\n"
-           "comparison = ("
-        << getFunctionName() << "(value) == " << computeExpectedResult(m_input) << ");\n";
+    oss << "result = " << getFunctionName() << "(value);\n"
+        << "comparison = (" << getFunctionName() << "(value) == " << computeExpectedResult(m_input) << ");\n";
     m_spec.source = oss.str();
 
     DE_ASSERT(!m_executor);
@@ -117,7 +115,7 @@ void UniformIntegerFunctionCase::init(void)
 tcu::TestNode::IterateResult UniformIntegerFunctionCase::iterate(void)
 {
     int32_t result;
-    bool comparison;
+    int32_t comparison; // A bool in the shader, but we must use a 32-bit type to copy out into.
     vector<void *> outputPointers(2);
 
     outputPointers[0] = &result;