From: Liam Middlebrook <lmiddlebrook@nvidia.com>
Date: Tue, 11 Sep 2018 05:42:26 +0000 (-0700)
Subject: Fix RobustnessVertexAccessInstance bounds check
X-Git-Tag: upstream/1.3.5~2324^2~20^2~21
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fed61233a802b5a8e7f6e634d0ef698e9155a94c;p=platform%2Fupstream%2FVK-GL-CTS.git

Fix RobustnessVertexAccessInstance bounds check

The bounds check preformed by
VertexAccessInstance::isValueWithinVertexBufferOrZero() needs to round
the value rather than truncating it when encoding it. When
`value * 0x3FFu` is casted to a deUint32 it's possible for the result to
be truncated instead of rounded causing unintended test failures.

Affects: dEQP-VK.robustness.vertex_access.a2b10g10r10_unorm_pack32.draw.vertex_incomplete

Components: Vulkan

VK-GL-CTS issue: 1351

Change-Id: Ib034b3e29fdbbbd4670e2bbdedad07492abf5cd4
---

diff --git a/external/vulkancts/modules/vulkan/robustness/vktRobustnessVertexAccessTests.cpp b/external/vulkancts/modules/vulkan/robustness/vktRobustnessVertexAccessTests.cpp
index 8a445dc..cc64ba1 100644
--- a/external/vulkancts/modules/vulkan/robustness/vktRobustnessVertexAccessTests.cpp
+++ b/external/vulkancts/modules/vulkan/robustness/vktRobustnessVertexAccessTests.cpp
@@ -899,9 +899,9 @@ bool VertexAccessInstance::isValueWithinVertexBufferOrZero(void* vertexBuffer, V
 		deUint32		encodedValue;
 
 		if (isAlpha)
-			encodedValue = deMin32(deUint32(normValue * 0x3u), 0x3u);
+			encodedValue = deMin32(deUint32(deFloatRound(normValue * 0x3u)), 0x3u);
 		else
-			encodedValue = deMin32(deUint32(normValue * 0x3FFu), 0x3FFu);
+			encodedValue = deMin32(deUint32(deFloatRound(normValue * 0x3FFu)), 0x3FFu);
 
 		if (encodedValue == 0)
 			return true;