Vulkan: fix validation failure when using fp16 feature
Although VkPhysicalDeviceVulkan11Features::storageBuffer16BitAccess is a subset of VkPhysicalDeviceVulkan11Features::uniformAndStorageBuffer16BitAccess, when the SPIR-V Capability (StorageBuffer16BitAccess) was declared, we must enable the VkPhysicalDeviceVulkan11Features::storageBuffer16BitAccess feature according to the requirements of the spec. BUG=dawn:675 Change-Id: I99f9eb4c5f2fd6c6565a51063817efb8bc88216f Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/42620 Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Hao Li <hao.x.li@intel.com> Reviewed-by: Jiawei Shao <jiawei.shao@intel.com> Commit-Queue: Jiawei Shao <jiawei.shao@intel.com>
This commit is contained in:
parent
e3f10e3d8e
commit
6cbef93c73
|
@ -87,6 +87,7 @@ namespace dawn_native { namespace vulkan {
|
|||
if (mDeviceInfo.HasExt(DeviceExt::ShaderFloat16Int8) &&
|
||||
mDeviceInfo.shaderFloat16Int8Features.shaderFloat16 == VK_TRUE &&
|
||||
mDeviceInfo.HasExt(DeviceExt::_16BitStorage) &&
|
||||
mDeviceInfo._16BitStorageFeatures.storageBuffer16BitAccess == VK_TRUE &&
|
||||
mDeviceInfo._16BitStorageFeatures.uniformAndStorageBuffer16BitAccess == VK_TRUE) {
|
||||
mSupportedExtensions.EnableExtension(Extension::ShaderFloat16);
|
||||
}
|
||||
|
|
|
@ -341,9 +341,11 @@ namespace dawn_native { namespace vulkan {
|
|||
ASSERT(deviceInfo.HasExt(DeviceExt::ShaderFloat16Int8) &&
|
||||
deviceInfo.shaderFloat16Int8Features.shaderFloat16 == VK_TRUE &&
|
||||
deviceInfo.HasExt(DeviceExt::_16BitStorage) &&
|
||||
deviceInfo._16BitStorageFeatures.storageBuffer16BitAccess == VK_TRUE &&
|
||||
deviceInfo._16BitStorageFeatures.uniformAndStorageBuffer16BitAccess == VK_TRUE);
|
||||
|
||||
usedKnobs.shaderFloat16Int8Features.shaderFloat16 = VK_TRUE;
|
||||
usedKnobs._16BitStorageFeatures.storageBuffer16BitAccess = VK_TRUE;
|
||||
usedKnobs._16BitStorageFeatures.uniformAndStorageBuffer16BitAccess = VK_TRUE;
|
||||
|
||||
featuresChain.Add(&usedKnobs.shaderFloat16Int8Features,
|
||||
|
|
Loading…
Reference in New Issue