tree ed3009d9cac3b19aadec17744d7fa6702c93a0a8
parent 1e0a2a7bfb01f1f138423f5e3466d184552f7674
author Ricardo Garcia <rgarcia@igalia.com> 1614006215 +0100
committer Alexander Galazin <Alexander.Galazin@arm.com> 1615812493 +0000

Always accumulate errors in desc set random tests

Tests using DEPENDENT and RUNTIME_SIZE index types were overwriting the
value of the accumulated error with each load operation. This has two
problems:

* Sometimes the result of a load operation would not be used in the next
  load operation. This happens when the next descriptor to check is not
  an array. In these cases the implementation may remove load
  operations, which means they would not be checked.

* As long as the implementation does not crash and the last load
  operation works as expected, the implementation was allowed to load
  incorrect values in those operations, because only the last load would
  be taken into account to decide if the output storage image will have
  a "good" or "bad" value.

This commit makes all test variants accumulate errors in the "accum"
variable that's used at the end of each test.

Affected tests:
dEQP-VK.binding_model.descriptorset_random.*

Components: Vulkan
VK-GL-CTS issue: 2763

Change-Id: I4de1bd4525d378b5ec2b7aad3faae81d956a9d7d