summaryrefslogtreecommitdiff
path: root/opengl/tests/gl_basic/gl_basic.cpp
diff options
context:
space:
mode:
authorJesse Hall <jessehall@google.com>2016-01-21 21:54:49 -0800
committerJesse Hall <jessehall@google.com>2016-01-25 13:49:31 -0800
commit9f763498058756b2c202d917075533142f86cdf0 (patch)
treee88b499213f515c8eae351259b39975dc74e0b6e /opengl/tests/gl_basic/gl_basic.cpp
parent26cecff368f2edba67d136ce00d27bb55634c0f1 (diff)
libvulkan: Fix dEQP-VK.api.object_management.alloc_callback_fail.device
In vkCreateDevice, we were using the app-provided allocator if available, otherwise the instance allocator. (The code also fell back to the default allocator if the instance didn't have an allocator, but it always does, so the fallback was dead code.) That's all as it should be. However, we don't store the app-provided allocator, and everywhere else just uses the instance allocator. In particular, vkDestroyDevice ignores the app-provided allocator, and just uses the instance allocator to free the Device structure. That causes the test to fail, since we use the app-provided device allocator to allocate, but the instance allocator to free. Getting this right will require some possibly large restructuring and careful review of the code. In the short term, this change just ignores the app-provided allocator in vkCreateDevice and uses the instance allocator instead, so that at least we use the same allocator for alloc and free. Bug 26732122 filed to do this properly. Change-Id: I4ed4c12dfa255bb58a33dca1a89946255276c791 (cherry picked from commit 58579313664f5d72706f8e3df50b4e7217f7326b)
Diffstat (limited to 'opengl/tests/gl_basic/gl_basic.cpp')
0 files changed, 0 insertions, 0 deletions