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