From c578741609cf57e83e1e13a260d7f081ff125245 Mon Sep 17 00:00:00 2001 From: Sivert Berg Date: Tue, 31 May 2016 14:37:57 +0200 Subject: [PATCH] Use TCU_CHECK instead of DE_ASSERT DE_ASSERT is optimized away in release build. In this case it resulted in the 'modifyXor' function not being called, causing the reference memory to go out of sync with the actual memory. (cherry picked from commit 8b9bed7d58e73412eeb361b612a51b89d1f30978) Bug: 29044372 Change-Id: I1cf5a5d756cc000dfa0d5fa55535865e13717442 --- .../vulkancts/modules/vulkan/memory/vktMemoryMappingTests.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/external/vulkancts/modules/vulkan/memory/vktMemoryMappingTests.cpp b/external/vulkancts/modules/vulkan/memory/vktMemoryMappingTests.cpp index 9bd06ecdc..ea650d539 100644 --- a/external/vulkancts/modules/vulkan/memory/vktMemoryMappingTests.cpp +++ b/external/vulkancts/modules/vulkan/memory/vktMemoryMappingTests.cpp @@ -556,7 +556,7 @@ void MemoryMapping::randomModify (de::Random& rng) const deUint8 mask = rng.getUint8(); ((deUint8*)m_ptr)[pos] = val ^ mask; - DE_ASSERT(m_reference.modifyXor((size_t)(m_range.offset + pos), val, mask)); + TCU_CHECK(m_reference.modifyXor((size_t)(m_range.offset + pos), val, mask)); } } -- 2.34.1