Validate instancing cases properly
authorIago Toral Quiroga <itoral@igalia.com>
Tue, 19 Dec 2017 12:30:14 +0000 (13:30 +0100)
committerAlexander Galazin <Alexander.Galazin@arm.com>
Mon, 22 Jan 2018 01:14:13 +0000 (20:14 -0500)
The vertex attrib 64bit limits test case executes
both regular and instanced draw calls and has a
validation function that takes a boolean parameter
that informs it about whether isntancing has been
used so it can verify that all instances executed
properly. However, this function is always called
with false as parameter, indicating no instancing,
even for instanced scenarios, which means that the
validation function is only validating the first
instance even for instanced cases.

Components: OpenGL
VK-GL-CTS issue: 916

Affects:
KHR-GL45.vertex_attrib_64bit.limits_test
KHR-GL46.vertex_attrib_64bit.limits_test

Change-Id: Ibaf2c8fc0b6599a17f7bdbc7c4914b42d38d5e2a

external/openglcts/modules/gl/gl4cVertexAttrib64BitTest.cpp

index 9d4c4f5..0b2823b 100644 (file)
@@ -2598,7 +2598,7 @@ bool LimitTest::testDrawArraysInstanced() const
        gl.endTransformFeedback();
        GLU_EXPECT_NO_ERROR(gl.getError(), "EndTransformFeedback");
 
-       if (true == verifyResult(false))
+       if (true == verifyResult(true))
        {
                return true;
        }
@@ -2652,7 +2652,7 @@ bool LimitTest::testDrawElementsInstanced() const
        gl.endTransformFeedback();
        GLU_EXPECT_NO_ERROR(gl.getError(), "EndTransformFeedback");
 
-       if (true == verifyResult(false))
+       if (true == verifyResult(true))
        {
                return true;
        }