Handle new device extension properties

Added new functions to explicitly mark some new extension features as not supported. Fixes the following test: dEQP-VK.api.info.get_physical_device_properties2.features Bug: b/144945388 Change-Id: Id4c5da51d42537f5e1d92352eece67d4e7f5b930 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38531 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by: Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by: Alexis Hétu <sugoi@google.com>

Handle new device extension properties
4584a9c2 · Alexis Hetu · Alexis Hétu · c8d4d425 · 4584a9c2
Commit 4584a9c2 authored Nov 21, 2019 by Alexis Hetu Committed by Alexis Hétu Nov 22, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 0 deletions

libVulkan.cpp src/Vulkan/libVulkan.cpp +8 -0

No files found.
--- a/src/Vulkan/libVulkan.cpp
+++ b/src/Vulkan/libVulkan.cpp
@@ -2581,6 +2581,14 @@ VKAPI_ATTR void VKAPI_CALL vkGetPhysicalDeviceFeatures2(VkPhysicalDevice physica
 			ASSERT(!HasExtensionProperty(VK_EXT_SCALAR_BLOCK_LAYOUT_EXTENSION_NAME, deviceExtensionProperties,
 										 sizeof(deviceExtensionProperties) / sizeof(deviceExtensionProperties[0])));
 			break;
+		case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_TIMELINE_SEMAPHORE_FEATURES_KHR:
+			ASSERT(!HasExtensionProperty(VK_KHR_TIMELINE_SEMAPHORE_EXTENSION_NAME, deviceExtensionProperties,
+										 sizeof(deviceExtensionProperties) / sizeof(deviceExtensionProperties[0])));
+			break;
+		case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_PERFORMANCE_QUERY_FEATURES_KHR:
+			ASSERT(!HasExtensionProperty(VK_KHR_PERFORMANCE_QUERY_EXTENSION_NAME, deviceExtensionProperties,
+										 sizeof(deviceExtensionProperties) / sizeof(deviceExtensionProperties[0])));
+			break;
 		default:
 			// "the [driver] must skip over, without processing (other than reading the sType and pNext members) any structures in the chain with sType values not defined by [supported extenions]"
 			UNIMPLEMENTED("extensionFeatures->sType");   // TODO(b/119321052): UNIMPLEMENTED() should be used only for features that must still be implemented. Use a more informational macro here.