Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2015 Google Inc. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license that can be |
| 5 | * found in the LICENSE file. |
| 6 | */ |
| 7 | |
| 8 | #include "GrVkCaps.h" |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 9 | #include "GrBackendSurface.h" |
Robert Phillips | bf25d43 | 2017-04-07 10:08:53 -0400 | [diff] [blame] | 10 | #include "GrRenderTargetProxy.h" |
Brian Salomon | 3d86a19 | 2018-02-27 16:46:11 -0500 | [diff] [blame] | 11 | #include "GrRenderTarget.h" |
Brian Salomon | 94efbf5 | 2016-11-29 13:43:05 -0500 | [diff] [blame] | 12 | #include "GrShaderCaps.h" |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 13 | #include "GrVkUtil.h" |
jvanverth | fd7bd45 | 2016-03-25 06:29:52 -0700 | [diff] [blame] | 14 | #include "vk/GrVkBackendContext.h" |
Brian Salomon | 467921e | 2017-03-06 16:17:12 -0500 | [diff] [blame] | 15 | #include "vk/GrVkInterface.h" |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 16 | |
| 17 | GrVkCaps::GrVkCaps(const GrContextOptions& contextOptions, const GrVkInterface* vkInterface, |
egdaniel | c5ec140 | 2016-03-28 12:14:42 -0700 | [diff] [blame] | 18 | VkPhysicalDevice physDev, uint32_t featureFlags, uint32_t extensionFlags) |
| 19 | : INHERITED(contextOptions) { |
| 20 | fCanUseGLSLForShaderModule = false; |
egdaniel | 6fa0a91 | 2016-09-12 11:51:29 -0700 | [diff] [blame] | 21 | fMustDoCopiesFromOrigin = false; |
Greg Daniel | 691f5e7 | 2018-02-28 14:21:34 -0500 | [diff] [blame] | 22 | fSupportsCopiesAsDraws = true; |
egdaniel | fd016d7 | 2016-09-27 12:13:05 -0700 | [diff] [blame] | 23 | fMustSubmitCommandsBeforeCopyOp = false; |
Greg Daniel | 80a08dd | 2017-01-20 10:45:49 -0500 | [diff] [blame] | 24 | fMustSleepOnTearDown = false; |
Greg Daniel | e3cd691 | 2017-05-17 11:15:55 -0400 | [diff] [blame] | 25 | fNewCBOnPipelineChange = false; |
Greg Daniel | 8385a8a | 2018-02-26 13:29:37 -0500 | [diff] [blame] | 26 | fCanUseWholeSizeOnFlushMappedMemory = true; |
egdaniel | c5ec140 | 2016-03-28 12:14:42 -0700 | [diff] [blame] | 27 | |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 28 | /************************************************************************** |
| 29 | * GrDrawTargetCaps fields |
| 30 | **************************************************************************/ |
jvanverth | 6234006 | 2016-04-26 08:01:44 -0700 | [diff] [blame] | 31 | fMipMapSupport = true; // always available in Vulkan |
brianosman | f05ab1b | 2016-05-12 11:01:10 -0700 | [diff] [blame] | 32 | fSRGBSupport = true; // always available in Vulkan |
Brian Osman | 57bc3ea | 2017-07-27 09:58:11 -0400 | [diff] [blame] | 33 | fSRGBDecodeDisableSupport = true; // always available in Vulkan |
brianosman | 8879186 | 2016-05-23 10:15:27 -0700 | [diff] [blame] | 34 | fNPOTTextureTileSupport = true; // always available in Vulkan |
egdaniel | 37535c9 | 2016-06-30 08:23:30 -0700 | [diff] [blame] | 35 | fDiscardRenderTargetSupport = true; |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 36 | fReuseScratchTextures = true; //TODO: figure this out |
| 37 | fGpuTracingSupport = false; //TODO: figure this out |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 38 | fOversizedStencilSupport = false; //TODO: figure this out |
Chris Dalton | 1d61635 | 2017-05-31 12:51:23 -0600 | [diff] [blame] | 39 | fInstanceAttribSupport = true; |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 40 | |
Chris Dalton | e4679fa | 2017-09-29 13:58:26 -0600 | [diff] [blame] | 41 | fBlacklistCoverageCounting = true; // blacklisting ccpr until we work through a few issues. |
jvanverth | 84741b3 | 2016-09-30 08:39:02 -0700 | [diff] [blame] | 42 | fFenceSyncSupport = true; // always available in Vulkan |
Greg Daniel | 691f5e7 | 2018-02-28 14:21:34 -0500 | [diff] [blame] | 43 | fCrossContextTextureSupport = true; |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 44 | |
| 45 | fMapBufferFlags = kNone_MapFlags; //TODO: figure this out |
cdalton | 397536c | 2016-03-25 12:15:03 -0700 | [diff] [blame] | 46 | fBufferMapThreshold = SK_MaxS32; //TODO: figure this out |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 47 | |
| 48 | fMaxRenderTargetSize = 4096; // minimum required by spec |
| 49 | fMaxTextureSize = 4096; // minimum required by spec |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 50 | |
Brian Salomon | 94efbf5 | 2016-11-29 13:43:05 -0500 | [diff] [blame] | 51 | fShaderCaps.reset(new GrShaderCaps(contextOptions)); |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 52 | |
egdaniel | c5ec140 | 2016-03-28 12:14:42 -0700 | [diff] [blame] | 53 | this->init(contextOptions, vkInterface, physDev, featureFlags, extensionFlags); |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 54 | } |
| 55 | |
Robert Phillips | bf25d43 | 2017-04-07 10:08:53 -0400 | [diff] [blame] | 56 | bool GrVkCaps::initDescForDstCopy(const GrRenderTargetProxy* src, GrSurfaceDesc* desc, |
Eric Karl | 7448088 | 2017-04-03 14:49:05 -0700 | [diff] [blame] | 57 | bool* rectsMustMatch, bool* disallowSubrect) const { |
| 58 | // Vk doesn't use rectsMustMatch or disallowSubrect. Always return false. |
| 59 | *rectsMustMatch = false; |
| 60 | *disallowSubrect = false; |
| 61 | |
Brian Salomon | 467921e | 2017-03-06 16:17:12 -0500 | [diff] [blame] | 62 | // We can always succeed here with either a CopyImage (none msaa src) or ResolveImage (msaa). |
| 63 | // For CopyImage we can make a simple texture, for ResolveImage we require the dst to be a |
| 64 | // render target as well. |
| 65 | desc->fOrigin = src->origin(); |
| 66 | desc->fConfig = src->config(); |
Robert Phillips | bf25d43 | 2017-04-07 10:08:53 -0400 | [diff] [blame] | 67 | if (src->numColorSamples() > 1 || (src->asTextureProxy() && this->supportsCopiesAsDraws())) { |
Brian Salomon | 467921e | 2017-03-06 16:17:12 -0500 | [diff] [blame] | 68 | desc->fFlags = kRenderTarget_GrSurfaceFlag; |
| 69 | } else { |
| 70 | // Just going to use CopyImage here |
| 71 | desc->fFlags = kNone_GrSurfaceFlags; |
| 72 | } |
| 73 | |
| 74 | return true; |
| 75 | } |
| 76 | |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 77 | void GrVkCaps::init(const GrContextOptions& contextOptions, const GrVkInterface* vkInterface, |
egdaniel | c5ec140 | 2016-03-28 12:14:42 -0700 | [diff] [blame] | 78 | VkPhysicalDevice physDev, uint32_t featureFlags, uint32_t extensionFlags) { |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 79 | |
Jim Van Verth | 6a40abc | 2017-11-02 16:56:09 +0000 | [diff] [blame] | 80 | VkPhysicalDeviceProperties properties; |
| 81 | GR_VK_CALL(vkInterface, GetPhysicalDeviceProperties(physDev, &properties)); |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 82 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 83 | VkPhysicalDeviceMemoryProperties memoryProperties; |
| 84 | GR_VK_CALL(vkInterface, GetPhysicalDeviceMemoryProperties(physDev, &memoryProperties)); |
| 85 | |
jvanverth | fd7bd45 | 2016-03-25 06:29:52 -0700 | [diff] [blame] | 86 | this->initGrCaps(properties, memoryProperties, featureFlags); |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 87 | this->initShaderCaps(properties, featureFlags); |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 88 | this->initConfigTable(vkInterface, physDev, properties); |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 89 | this->initStencilFormat(vkInterface, physDev); |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 90 | |
Greg Daniel | 691f5e7 | 2018-02-28 14:21:34 -0500 | [diff] [blame] | 91 | if (!contextOptions.fDisableDriverCorrectnessWorkarounds) { |
| 92 | this->applyDriverCorrectnessWorkarounds(properties); |
egdaniel | c5ec140 | 2016-03-28 12:14:42 -0700 | [diff] [blame] | 93 | } |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 94 | |
Greg Daniel | 691f5e7 | 2018-02-28 14:21:34 -0500 | [diff] [blame] | 95 | this->applyOptionsOverrides(contextOptions); |
| 96 | fShaderCaps->applyOptionsOverrides(contextOptions); |
| 97 | } |
| 98 | |
| 99 | void GrVkCaps::applyDriverCorrectnessWorkarounds(const VkPhysicalDeviceProperties& properties) { |
egdaniel | 6fa0a91 | 2016-09-12 11:51:29 -0700 | [diff] [blame] | 100 | if (kQualcomm_VkVendor == properties.vendorID) { |
| 101 | fMustDoCopiesFromOrigin = true; |
| 102 | } |
| 103 | |
egdaniel | fd016d7 | 2016-09-27 12:13:05 -0700 | [diff] [blame] | 104 | if (kNvidia_VkVendor == properties.vendorID) { |
egdaniel | fd016d7 | 2016-09-27 12:13:05 -0700 | [diff] [blame] | 105 | fMustSubmitCommandsBeforeCopyOp = true; |
| 106 | } |
| 107 | |
Greg Daniel | 691f5e7 | 2018-02-28 14:21:34 -0500 | [diff] [blame] | 108 | if (kQualcomm_VkVendor == properties.vendorID || |
| 109 | kARM_VkVendor == properties.vendorID) { |
| 110 | fSupportsCopiesAsDraws = false; |
| 111 | // We require copies as draws to support cross context textures. |
| 112 | fCrossContextTextureSupport = false; |
Greg Daniel | a63e7ab | 2017-05-16 14:38:54 -0400 | [diff] [blame] | 113 | } |
| 114 | |
Greg Daniel | 80a08dd | 2017-01-20 10:45:49 -0500 | [diff] [blame] | 115 | #if defined(SK_BUILD_FOR_WIN) |
| 116 | if (kNvidia_VkVendor == properties.vendorID) { |
| 117 | fMustSleepOnTearDown = true; |
| 118 | } |
| 119 | #elif defined(SK_BUILD_FOR_ANDROID) |
| 120 | if (kImagination_VkVendor == properties.vendorID) { |
| 121 | fMustSleepOnTearDown = true; |
| 122 | } |
| 123 | #endif |
Greg Daniel | bce5eb9 | 2018-03-01 13:13:44 -0500 | [diff] [blame^] | 124 | |
| 125 | // AMD seems to have issues binding new VkPipelines inside a secondary command buffer. |
| 126 | // Current workaround is to use a different secondary command buffer for each new VkPipeline. |
| 127 | if (kAMD_VkVendor == properties.vendorID) { |
| 128 | fNewCBOnPipelineChange = true; |
| 129 | } |
| 130 | |
| 131 | //////////////////////////////////////////////////////////////////////////// |
| 132 | // GrCaps workarounds |
| 133 | //////////////////////////////////////////////////////////////////////////// |
| 134 | |
| 135 | if (kARM_VkVendor == properties.vendorID) { |
| 136 | fInstanceAttribSupport = false; |
| 137 | } |
| 138 | |
| 139 | // AMD advertises support for MAX_UINT vertex input attributes, but in reality only supports 32. |
| 140 | if (kAMD_VkVendor == properties.vendorID) { |
| 141 | fMaxVertexAttributes = SkTMin(fMaxVertexAttributes, 32); |
| 142 | } |
| 143 | |
| 144 | if (kIntel_VkVendor == properties.vendorID) { |
| 145 | fCanUseWholeSizeOnFlushMappedMemory = false; |
| 146 | } |
| 147 | |
| 148 | #if defined(SK_CPU_X86) |
| 149 | if (kImagination_VkVendor == properties.vendorID) { |
| 150 | fSRGBSupport = false; |
| 151 | } |
| 152 | #endif |
| 153 | |
| 154 | //////////////////////////////////////////////////////////////////////////// |
| 155 | // GrShaderCaps workarounds |
| 156 | //////////////////////////////////////////////////////////////////////////// |
| 157 | |
| 158 | if (kAMD_VkVendor == properties.vendorID) { |
| 159 | // Currently DualSourceBlending is not working on AMD. vkCreateGraphicsPipeline fails when |
| 160 | // using a draw with dual source. Looking into whether it is driver bug or issue with our |
| 161 | // SPIR-V. Bug skia:6405 |
| 162 | fShaderCaps->fDualSourceBlendingSupport = false; |
| 163 | } |
| 164 | |
| 165 | if (kImagination_VkVendor == properties.vendorID) { |
| 166 | fShaderCaps->fAtan2ImplementedAsAtanYOverX = true; |
| 167 | } |
| 168 | |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 169 | } |
| 170 | |
| 171 | int get_max_sample_count(VkSampleCountFlags flags) { |
| 172 | SkASSERT(flags & VK_SAMPLE_COUNT_1_BIT); |
| 173 | if (!(flags & VK_SAMPLE_COUNT_2_BIT)) { |
| 174 | return 0; |
| 175 | } |
| 176 | if (!(flags & VK_SAMPLE_COUNT_4_BIT)) { |
| 177 | return 2; |
| 178 | } |
| 179 | if (!(flags & VK_SAMPLE_COUNT_8_BIT)) { |
| 180 | return 4; |
| 181 | } |
| 182 | if (!(flags & VK_SAMPLE_COUNT_16_BIT)) { |
| 183 | return 8; |
| 184 | } |
| 185 | if (!(flags & VK_SAMPLE_COUNT_32_BIT)) { |
| 186 | return 16; |
| 187 | } |
| 188 | if (!(flags & VK_SAMPLE_COUNT_64_BIT)) { |
| 189 | return 32; |
| 190 | } |
| 191 | return 64; |
| 192 | } |
| 193 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 194 | void GrVkCaps::initGrCaps(const VkPhysicalDeviceProperties& properties, |
jvanverth | fd7bd45 | 2016-03-25 06:29:52 -0700 | [diff] [blame] | 195 | const VkPhysicalDeviceMemoryProperties& memoryProperties, |
| 196 | uint32_t featureFlags) { |
Greg Daniel | c5cc2de | 2017-03-20 11:40:58 -0400 | [diff] [blame] | 197 | // So GPUs, like AMD, are reporting MAX_INT support vertex attributes. In general, there is no |
| 198 | // need for us ever to support that amount, and it makes tests which tests all the vertex |
| 199 | // attribs timeout looping over that many. For now, we'll cap this at 64 max and can raise it if |
| 200 | // we ever find that need. |
| 201 | static const uint32_t kMaxVertexAttributes = 64; |
| 202 | fMaxVertexAttributes = SkTMin(properties.limits.maxVertexInputAttributes, kMaxVertexAttributes); |
Greg Daniel | c5cc2de | 2017-03-20 11:40:58 -0400 | [diff] [blame] | 203 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 204 | // We could actually query and get a max size for each config, however maxImageDimension2D will |
| 205 | // give the minimum max size across all configs. So for simplicity we will use that for now. |
jvanverth | e78d487 | 2016-09-27 03:33:05 -0700 | [diff] [blame] | 206 | fMaxRenderTargetSize = SkTMin(properties.limits.maxImageDimension2D, (uint32_t)INT_MAX); |
| 207 | fMaxTextureSize = SkTMin(properties.limits.maxImageDimension2D, (uint32_t)INT_MAX); |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 208 | |
Chris Dalton | 2612bae | 2018-02-22 13:41:37 -0700 | [diff] [blame] | 209 | // TODO: check if RT's larger than 4k incur a performance cost on ARM. |
| 210 | fMaxPreferredRenderTargetSize = fMaxRenderTargetSize; |
| 211 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 212 | // Assuming since we will always map in the end to upload the data we might as well just map |
| 213 | // from the get go. There is no hard data to suggest this is faster or slower. |
cdalton | 397536c | 2016-03-25 12:15:03 -0700 | [diff] [blame] | 214 | fBufferMapThreshold = 0; |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 215 | |
| 216 | fMapBufferFlags = kCanMap_MapFlag | kSubset_MapFlag; |
| 217 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 218 | fOversizedStencilSupport = true; |
ethannicholas | 28ef445 | 2016-03-25 09:26:03 -0700 | [diff] [blame] | 219 | fSampleShadingSupport = SkToBool(featureFlags & kSampleRateShading_GrVkFeatureFlag); |
Greg Daniel | 22bc865 | 2017-03-22 15:45:43 -0400 | [diff] [blame] | 220 | |
Greg Daniel | 0190787 | 2017-05-23 15:21:02 -0400 | [diff] [blame] | 221 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 222 | } |
| 223 | |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 224 | void GrVkCaps::initShaderCaps(const VkPhysicalDeviceProperties& properties, uint32_t featureFlags) { |
| 225 | GrShaderCaps* shaderCaps = fShaderCaps.get(); |
| 226 | shaderCaps->fVersionDeclString = "#version 330\n"; |
egdaniel | 3a15fd4 | 2016-04-05 11:00:29 -0700 | [diff] [blame] | 227 | |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 228 | |
| 229 | // fConfigOutputSwizzle will default to RGBA so we only need to set it for alpha only config. |
| 230 | for (int i = 0; i < kGrPixelConfigCnt; ++i) { |
| 231 | GrPixelConfig config = static_cast<GrPixelConfig>(i); |
Greg Daniel | ef59d87 | 2017-11-17 16:47:21 -0500 | [diff] [blame] | 232 | // Vulkan doesn't support a single channel format stored in alpha. |
| 233 | if (GrPixelConfigIsAlphaOnly(config) && |
| 234 | kAlpha_8_as_Alpha_GrPixelConfig != config) { |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 235 | shaderCaps->fConfigTextureSwizzle[i] = GrSwizzle::RRRR(); |
| 236 | shaderCaps->fConfigOutputSwizzle[i] = GrSwizzle::AAAA(); |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 237 | } else { |
Greg Daniel | 7af060a | 2017-12-05 16:27:11 -0500 | [diff] [blame] | 238 | if (kGray_8_GrPixelConfig == config || |
| 239 | kGray_8_as_Red_GrPixelConfig == config) { |
Brian Osman | 986563b | 2017-01-10 14:20:02 -0500 | [diff] [blame] | 240 | shaderCaps->fConfigTextureSwizzle[i] = GrSwizzle::RRRA(); |
| 241 | } else if (kRGBA_4444_GrPixelConfig == config) { |
egdaniel | 3fe0327 | 2016-08-15 10:59:17 -0700 | [diff] [blame] | 242 | // The vulkan spec does not require R4G4B4A4 to be supported for texturing so we |
| 243 | // store the data in a B4G4R4A4 texture and then swizzle it when doing texture reads |
| 244 | // or writing to outputs. Since we're not actually changing the data at all, the |
| 245 | // only extra work is the swizzle in the shader for all operations. |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 246 | shaderCaps->fConfigTextureSwizzle[i] = GrSwizzle::BGRA(); |
| 247 | shaderCaps->fConfigOutputSwizzle[i] = GrSwizzle::BGRA(); |
egdaniel | 3fe0327 | 2016-08-15 10:59:17 -0700 | [diff] [blame] | 248 | } else { |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 249 | shaderCaps->fConfigTextureSwizzle[i] = GrSwizzle::RGBA(); |
egdaniel | 3fe0327 | 2016-08-15 10:59:17 -0700 | [diff] [blame] | 250 | } |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 251 | } |
| 252 | } |
egdaniel | 67acb83 | 2016-02-26 08:32:20 -0800 | [diff] [blame] | 253 | |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 254 | // Vulkan is based off ES 3.0 so the following should all be supported |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 255 | shaderCaps->fUsesPrecisionModifiers = true; |
| 256 | shaderCaps->fFlatInterpolationSupport = true; |
Brian Salomon | 4127456 | 2017-09-15 09:40:03 -0700 | [diff] [blame] | 257 | // Flat interpolation appears to be slow on Qualcomm GPUs. This was tested in GL and is assumed |
| 258 | // to be true with Vulkan as well. |
| 259 | shaderCaps->fPreferFlatInterpolation = kQualcomm_VkVendor != properties.vendorID; |
egdaniel | d5e3b9e | 2016-03-08 12:19:54 -0800 | [diff] [blame] | 260 | |
| 261 | // GrShaderCaps |
| 262 | |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 263 | shaderCaps->fShaderDerivativeSupport = true; |
Chris Dalton | f1b47bb | 2017-10-06 11:57:51 -0600 | [diff] [blame] | 264 | |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 265 | shaderCaps->fGeometryShaderSupport = SkToBool(featureFlags & kGeometryShader_GrVkFeatureFlag); |
Chris Dalton | f1b47bb | 2017-10-06 11:57:51 -0600 | [diff] [blame] | 266 | shaderCaps->fGSInvocationsSupport = shaderCaps->fGeometryShaderSupport; |
egdaniel | d632bb4 | 2016-03-30 12:06:48 -0700 | [diff] [blame] | 267 | |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 268 | shaderCaps->fDualSourceBlendingSupport = SkToBool(featureFlags & kDualSrcBlend_GrVkFeatureFlag); |
egdaniel | d632bb4 | 2016-03-30 12:06:48 -0700 | [diff] [blame] | 269 | |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 270 | shaderCaps->fIntegerSupport = true; |
Chris Dalton | 6b65b98 | 2017-07-06 11:04:00 -0600 | [diff] [blame] | 271 | shaderCaps->fTexelBufferSupport = true; |
| 272 | shaderCaps->fTexelFetchSupport = true; |
Chris Dalton | 1d61635 | 2017-05-31 12:51:23 -0600 | [diff] [blame] | 273 | shaderCaps->fVertexIDSupport = true; |
cdalton | 9c3f143 | 2016-03-11 10:07:37 -0800 | [diff] [blame] | 274 | |
cdalton | a6b92ad | 2016-04-11 12:03:08 -0700 | [diff] [blame] | 275 | // Assume the minimum precisions mandated by the SPIR-V spec. |
Chris Dalton | 47c8ed3 | 2017-11-15 18:27:09 -0700 | [diff] [blame] | 276 | shaderCaps->fFloatIs32Bits = true; |
| 277 | shaderCaps->fHalfIs32Bits = false; |
cdalton | a6b92ad | 2016-04-11 12:03:08 -0700 | [diff] [blame] | 278 | |
Brian Salomon | 1edc5b9 | 2016-11-29 13:43:46 -0500 | [diff] [blame] | 279 | shaderCaps->fMaxVertexSamplers = |
| 280 | shaderCaps->fMaxGeometrySamplers = |
| 281 | shaderCaps->fMaxFragmentSamplers = SkTMin( |
| 282 | SkTMin(properties.limits.maxPerStageDescriptorSampledImages, |
| 283 | properties.limits.maxPerStageDescriptorSamplers), |
| 284 | (uint32_t)INT_MAX); |
| 285 | shaderCaps->fMaxCombinedSamplers = SkTMin( |
| 286 | SkTMin(properties.limits.maxDescriptorSetSampledImages, |
| 287 | properties.limits.maxDescriptorSetSamplers), |
| 288 | (uint32_t)INT_MAX); |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 289 | } |
| 290 | |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 291 | bool stencil_format_supported(const GrVkInterface* interface, |
| 292 | VkPhysicalDevice physDev, |
| 293 | VkFormat format) { |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 294 | VkFormatProperties props; |
| 295 | memset(&props, 0, sizeof(VkFormatProperties)); |
| 296 | GR_VK_CALL(interface, GetPhysicalDeviceFormatProperties(physDev, format, &props)); |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 297 | return SkToBool(VK_FORMAT_FEATURE_DEPTH_STENCIL_ATTACHMENT_BIT & props.optimalTilingFeatures); |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 298 | } |
| 299 | |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 300 | void GrVkCaps::initStencilFormat(const GrVkInterface* interface, VkPhysicalDevice physDev) { |
| 301 | // List of legal stencil formats (though perhaps not supported on |
| 302 | // the particular gpu/driver) from most preferred to least. We are guaranteed to have either |
jvanverth | a4b0fed | 2016-04-27 11:42:21 -0700 | [diff] [blame] | 303 | // VK_FORMAT_D24_UNORM_S8_UINT or VK_FORMAT_D32_SFLOAT_S8_UINT. VK_FORMAT_D32_SFLOAT_S8_UINT |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 304 | // can optionally have 24 unused bits at the end so we assume the total bits is 64. |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 305 | static const StencilFormat |
| 306 | // internal Format stencil bits total bits packed? |
| 307 | gS8 = { VK_FORMAT_S8_UINT, 8, 8, false }, |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 308 | gD24S8 = { VK_FORMAT_D24_UNORM_S8_UINT, 8, 32, true }, |
| 309 | gD32S8 = { VK_FORMAT_D32_SFLOAT_S8_UINT, 8, 64, true }; |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 310 | |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 311 | if (stencil_format_supported(interface, physDev, VK_FORMAT_S8_UINT)) { |
| 312 | fPreferedStencilFormat = gS8; |
| 313 | } else if (stencil_format_supported(interface, physDev, VK_FORMAT_D24_UNORM_S8_UINT)) { |
| 314 | fPreferedStencilFormat = gD24S8; |
| 315 | } else { |
| 316 | SkASSERT(stencil_format_supported(interface, physDev, VK_FORMAT_D32_SFLOAT_S8_UINT)); |
| 317 | fPreferedStencilFormat = gD32S8; |
| 318 | } |
| 319 | } |
| 320 | |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 321 | void GrVkCaps::initConfigTable(const GrVkInterface* interface, VkPhysicalDevice physDev, |
| 322 | const VkPhysicalDeviceProperties& properties) { |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 323 | for (int i = 0; i < kGrPixelConfigCnt; ++i) { |
| 324 | VkFormat format; |
| 325 | if (GrPixelConfigToVkFormat(static_cast<GrPixelConfig>(i), &format)) { |
Greg Daniel | 0190787 | 2017-05-23 15:21:02 -0400 | [diff] [blame] | 326 | if (!GrPixelConfigIsSRGB(static_cast<GrPixelConfig>(i)) || fSRGBSupport) { |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 327 | fConfigTable[i].init(interface, physDev, properties, format); |
Greg Daniel | 0190787 | 2017-05-23 15:21:02 -0400 | [diff] [blame] | 328 | } |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 329 | } |
| 330 | } |
| 331 | } |
| 332 | |
| 333 | void GrVkCaps::ConfigInfo::InitConfigFlags(VkFormatFeatureFlags vkFlags, uint16_t* flags) { |
| 334 | if (SkToBool(VK_FORMAT_FEATURE_SAMPLED_IMAGE_BIT & vkFlags) && |
| 335 | SkToBool(VK_FORMAT_FEATURE_SAMPLED_IMAGE_FILTER_LINEAR_BIT & vkFlags)) { |
| 336 | *flags = *flags | kTextureable_Flag; |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 337 | |
Robert Phillips | b7b7e5f | 2017-05-22 13:23:19 -0400 | [diff] [blame] | 338 | // Ganesh assumes that all renderable surfaces are also texturable |
| 339 | if (SkToBool(VK_FORMAT_FEATURE_COLOR_ATTACHMENT_BLEND_BIT & vkFlags)) { |
| 340 | *flags = *flags | kRenderable_Flag; |
| 341 | } |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 342 | } |
| 343 | |
| 344 | if (SkToBool(VK_FORMAT_FEATURE_BLIT_SRC_BIT & vkFlags)) { |
| 345 | *flags = *flags | kBlitSrc_Flag; |
| 346 | } |
| 347 | |
| 348 | if (SkToBool(VK_FORMAT_FEATURE_BLIT_DST_BIT & vkFlags)) { |
| 349 | *flags = *flags | kBlitDst_Flag; |
| 350 | } |
| 351 | } |
| 352 | |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 353 | void GrVkCaps::ConfigInfo::initSampleCounts(const GrVkInterface* interface, |
| 354 | VkPhysicalDevice physDev, |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 355 | const VkPhysicalDeviceProperties& physProps, |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 356 | VkFormat format) { |
| 357 | VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | |
| 358 | VK_IMAGE_USAGE_TRANSFER_DST_BIT | |
| 359 | VK_IMAGE_USAGE_SAMPLED_BIT | |
| 360 | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT; |
| 361 | VkImageCreateFlags createFlags = GrVkFormatIsSRGB(format, nullptr) |
| 362 | ? VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT : 0; |
| 363 | VkImageFormatProperties properties; |
| 364 | GR_VK_CALL(interface, GetPhysicalDeviceImageFormatProperties(physDev, |
| 365 | format, |
| 366 | VK_IMAGE_TYPE_2D, |
| 367 | VK_IMAGE_TILING_OPTIMAL, |
| 368 | usage, |
| 369 | createFlags, |
| 370 | &properties)); |
| 371 | VkSampleCountFlags flags = properties.sampleCounts; |
| 372 | if (flags & VK_SAMPLE_COUNT_1_BIT) { |
Brian Salomon | bdecacf | 2018-02-02 20:32:49 -0500 | [diff] [blame] | 373 | fColorSampleCounts.push(1); |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 374 | } |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 375 | if (kImagination_VkVendor == physProps.vendorID) { |
| 376 | // MSAA does not work on imagination |
| 377 | return; |
| 378 | } |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 379 | if (flags & VK_SAMPLE_COUNT_2_BIT) { |
| 380 | fColorSampleCounts.push(2); |
| 381 | } |
| 382 | if (flags & VK_SAMPLE_COUNT_4_BIT) { |
| 383 | fColorSampleCounts.push(4); |
| 384 | } |
| 385 | if (flags & VK_SAMPLE_COUNT_8_BIT) { |
| 386 | fColorSampleCounts.push(8); |
| 387 | } |
| 388 | if (flags & VK_SAMPLE_COUNT_16_BIT) { |
| 389 | fColorSampleCounts.push(16); |
| 390 | } |
| 391 | if (flags & VK_SAMPLE_COUNT_32_BIT) { |
| 392 | fColorSampleCounts.push(32); |
| 393 | } |
| 394 | if (flags & VK_SAMPLE_COUNT_64_BIT) { |
| 395 | fColorSampleCounts.push(64); |
| 396 | } |
| 397 | } |
| 398 | |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 399 | void GrVkCaps::ConfigInfo::init(const GrVkInterface* interface, |
| 400 | VkPhysicalDevice physDev, |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 401 | const VkPhysicalDeviceProperties& properties, |
egdaniel | 8f1dcaa | 2016-04-01 10:10:45 -0700 | [diff] [blame] | 402 | VkFormat format) { |
| 403 | VkFormatProperties props; |
| 404 | memset(&props, 0, sizeof(VkFormatProperties)); |
| 405 | GR_VK_CALL(interface, GetPhysicalDeviceFormatProperties(physDev, format, &props)); |
| 406 | InitConfigFlags(props.linearTilingFeatures, &fLinearFlags); |
| 407 | InitConfigFlags(props.optimalTilingFeatures, &fOptimalFlags); |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 408 | if (fOptimalFlags & kRenderable_Flag) { |
Greg Daniel | 2bb6ecc | 2017-07-20 13:11:14 +0000 | [diff] [blame] | 409 | this->initSampleCounts(interface, physDev, properties, format); |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 410 | } |
Greg Daniel | 164a9f0 | 2016-02-22 09:56:40 -0500 | [diff] [blame] | 411 | } |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 412 | |
Brian Salomon | bdecacf | 2018-02-02 20:32:49 -0500 | [diff] [blame] | 413 | int GrVkCaps::getRenderTargetSampleCount(int requestedCount, GrPixelConfig config) const { |
| 414 | requestedCount = SkTMax(1, requestedCount); |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 415 | int count = fConfigTable[config].fColorSampleCounts.count(); |
Brian Salomon | bdecacf | 2018-02-02 20:32:49 -0500 | [diff] [blame] | 416 | |
| 417 | if (!count) { |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 418 | return 0; |
| 419 | } |
| 420 | |
Brian Salomon | bdecacf | 2018-02-02 20:32:49 -0500 | [diff] [blame] | 421 | if (1 == requestedCount) { |
| 422 | SkASSERT(fConfigTable[config].fColorSampleCounts.count() && |
| 423 | fConfigTable[config].fColorSampleCounts[0] == 1); |
| 424 | return 1; |
| 425 | } |
| 426 | |
Greg Daniel | 81e7bf8 | 2017-07-19 14:47:42 -0400 | [diff] [blame] | 427 | for (int i = 0; i < count; ++i) { |
| 428 | if (fConfigTable[config].fColorSampleCounts[i] >= requestedCount) { |
| 429 | return fConfigTable[config].fColorSampleCounts[i]; |
| 430 | } |
| 431 | } |
Brian Salomon | bdecacf | 2018-02-02 20:32:49 -0500 | [diff] [blame] | 432 | return 0; |
| 433 | } |
| 434 | |
| 435 | int GrVkCaps::maxRenderTargetSampleCount(GrPixelConfig config) const { |
| 436 | const auto& table = fConfigTable[config].fColorSampleCounts; |
| 437 | if (!table.count()) { |
| 438 | return 0; |
| 439 | } |
| 440 | return table[table.count() - 1]; |
Brian Salomon | d653cac | 2018-02-01 13:58:00 -0500 | [diff] [blame] | 441 | } |
| 442 | |
Brian Salomon | 3d86a19 | 2018-02-27 16:46:11 -0500 | [diff] [blame] | 443 | bool GrVkCaps::surfaceSupportsWritePixels(const GrSurface* surface) const { |
| 444 | if (auto rt = surface->asRenderTarget()) { |
| 445 | return rt->numColorSamples() <= 1 && SkToBool(surface->asTexture()); |
| 446 | } |
| 447 | return true; |
| 448 | } |
| 449 | |
Robert Phillips | fc711a2 | 2018-02-13 17:03:00 -0500 | [diff] [blame] | 450 | bool validate_image_info(VkFormat format, SkColorType ct, GrPixelConfig* config) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 451 | *config = kUnknown_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 452 | |
| 453 | switch (ct) { |
| 454 | case kUnknown_SkColorType: |
| 455 | return false; |
| 456 | case kAlpha_8_SkColorType: |
| 457 | if (VK_FORMAT_R8_UNORM == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 458 | *config = kAlpha_8_as_Red_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 459 | } |
| 460 | break; |
| 461 | case kRGB_565_SkColorType: |
| 462 | if (VK_FORMAT_R5G6B5_UNORM_PACK16 == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 463 | *config = kRGB_565_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 464 | } |
| 465 | break; |
| 466 | case kARGB_4444_SkColorType: |
| 467 | if (VK_FORMAT_B4G4R4A4_UNORM_PACK16 == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 468 | *config = kRGBA_4444_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 469 | } |
| 470 | break; |
| 471 | case kRGBA_8888_SkColorType: |
| 472 | if (VK_FORMAT_R8G8B8A8_UNORM == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 473 | *config = kRGBA_8888_GrPixelConfig; |
Greg Daniel | 7b219ac | 2017-12-18 14:49:04 -0500 | [diff] [blame] | 474 | } else if (VK_FORMAT_R8G8B8A8_SRGB == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 475 | *config = kSRGBA_8888_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 476 | } |
| 477 | break; |
Brian Salomon | e41e176 | 2018-01-25 14:07:47 -0500 | [diff] [blame] | 478 | case kRGB_888x_SkColorType: |
| 479 | return false; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 480 | case kBGRA_8888_SkColorType: |
| 481 | if (VK_FORMAT_B8G8R8A8_UNORM == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 482 | *config = kBGRA_8888_GrPixelConfig; |
Greg Daniel | 7b219ac | 2017-12-18 14:49:04 -0500 | [diff] [blame] | 483 | } else if (VK_FORMAT_B8G8R8A8_SRGB == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 484 | *config = kSBGRA_8888_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 485 | } |
| 486 | break; |
Brian Salomon | e41e176 | 2018-01-25 14:07:47 -0500 | [diff] [blame] | 487 | case kRGBA_1010102_SkColorType: |
Brian Osman | 44b6120 | 2018-03-01 10:49:26 -0500 | [diff] [blame] | 488 | if (VK_FORMAT_A2B10G10R10_UNORM_PACK32 == format) { |
| 489 | *config = kRGBA_1010102_GrPixelConfig; |
| 490 | } |
| 491 | break; |
Brian Salomon | e41e176 | 2018-01-25 14:07:47 -0500 | [diff] [blame] | 492 | case kRGB_101010x_SkColorType: |
| 493 | return false; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 494 | case kGray_8_SkColorType: |
| 495 | if (VK_FORMAT_R8_UNORM == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 496 | *config = kGray_8_as_Red_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 497 | } |
| 498 | break; |
| 499 | case kRGBA_F16_SkColorType: |
| 500 | if (VK_FORMAT_R16G16B16A16_SFLOAT == format) { |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 501 | *config = kRGBA_half_GrPixelConfig; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 502 | } |
| 503 | break; |
| 504 | } |
| 505 | |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 506 | return kUnknown_GrPixelConfig != *config; |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 507 | } |
| 508 | |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 509 | bool GrVkCaps::validateBackendTexture(const GrBackendTexture& tex, SkColorType ct, |
| 510 | GrPixelConfig* config) const { |
Robert Phillips | fc711a2 | 2018-02-13 17:03:00 -0500 | [diff] [blame] | 511 | const GrVkImageInfo* imageInfo = tex.getVkImageInfo(); |
| 512 | if (!imageInfo) { |
| 513 | return false; |
| 514 | } |
| 515 | |
| 516 | return validate_image_info(imageInfo->fFormat, ct, config); |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 517 | } |
| 518 | |
| 519 | bool GrVkCaps::validateBackendRenderTarget(const GrBackendRenderTarget& rt, SkColorType ct, |
| 520 | GrPixelConfig* config) const { |
Robert Phillips | fc711a2 | 2018-02-13 17:03:00 -0500 | [diff] [blame] | 521 | const GrVkImageInfo* imageInfo = rt.getVkImageInfo(); |
| 522 | if (!imageInfo) { |
| 523 | return false; |
| 524 | } |
| 525 | |
| 526 | return validate_image_info(imageInfo->fFormat, ct, config); |
| 527 | } |
| 528 | |
| 529 | bool GrVkCaps::getConfigFromBackendFormat(const GrBackendFormat& format, SkColorType ct, |
| 530 | GrPixelConfig* config) const { |
| 531 | const VkFormat* vkFormat = format.getVkFormat(); |
| 532 | if (!vkFormat) { |
| 533 | return false; |
| 534 | } |
| 535 | return validate_image_info(*vkFormat, ct, config); |
Greg Daniel | faa095e | 2017-12-19 13:15:02 -0500 | [diff] [blame] | 536 | } |
Greg Daniel | f5d8758 | 2017-12-18 14:48:15 -0500 | [diff] [blame] | 537 | |