Use TCU_CHECK instead of DE_ASSERT
authorSivert Berg <sivert.berg@arm.com>
Tue, 31 May 2016 12:37:57 +0000 (14:37 +0200)
committerPyry Haulos <phaulos@google.com>
Tue, 31 May 2016 16:52:07 +0000 (09:52 -0700)
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

external/vulkancts/modules/vulkan/memory/vktMemoryMappingTests.cpp

index 9bd06ecdc3b7cbd3cc9af99ae806a9908d41e3f0..ea650d539b1acf022ae112cc375020079dd8d8c3 100644 (file)
@@ -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));
        }
 }