2500 Commits

Author SHA1 Message Date
Bill Hollings
be6f7a5083 Update dependency libraries to match Vulkan SDK 1.3.211.
Update What's New document.
2022-04-09 16:28:41 -04:00
Bill Hollings
2a565ef8e7
Merge pull request #1566 from billhollings/VK_EXT_sample_locations
Add support for VK_EXT_sample_locations extension.
2022-04-08 21:05:52 -04:00
Bill Hollings
3c0644f36a Add support for VK_EXT_sample_locations extension.
Supports only setting custom sample locations in subpasses via
vkBeginRenderpass. Does not support setting custom sample locations via
vkCmdBindPipeline or vkCmdSetSampleLocationsEXT, although collects that
info for possible future enhancements.

- MVKPhysicalDevice track platform support and respond to property queries.
- MVKCmdBeginRenderPassBase collect subpass custom sample locations.
- MVKPipeline support dynamic state values beyond 31.
- MVKPipeline collect custom sample locations.
- Add MVKCmdSetSampleLocations to support vkCmdSetSampleLocations
  to collect dynamic custom sample locations.
- MVKCommandEncoder support collecting custom sample positions from subpass
  and dynamic, and set into MTLRenderPassDescriptor for each Metal render pass.
- MVKArrayRef add assignment operator.
- Add MVKPhysicalDeviceMetalFeatures::programmableSamplePositions.
- Update VK_MVK_MOLTENVK_SPEC_VERSION to version 34.
- MVKCommandBuffer.h remove obsolete comment documentation.
- Update Whats_New.md.
2022-04-08 18:50:32 -04:00
Bill Hollings
148823a841
Merge pull request #1564 from bangnoise/pr/IOSurface-CF-leaks
Fix leak of CoreFoundation objects during calls to vkUseIOSurfaceMVK()
2022-04-08 13:06:05 -04:00
Bill Hollings
8ad7ee8984
Merge pull request #1563 from bangnoise/pr/vkGetMTLCommandQueueMVK
Fix crash in vkGetMTLCommandQueueMVK()
2022-04-08 13:05:47 -04:00
Tom Butterworth
9f5bd82bef Fix leak of CoreFoundation objects during calls to vkUseIOSurfaceMVK()
A small leak occurs if no existing IOSurface is provided to vkUseIOSurfaceMVK() because CoreFoundation objects returned from functions with Create in their name must be released with CFRelease()
2022-04-08 10:44:13 +01:00
Tom Butterworth
41fabde757 Fix crash in vkGetMTLCommandQueueMVK()
MVKQueue* cannot be cast from VkQueue, use MVKQueue::getMVKQueue() instead
2022-04-08 10:12:21 +01:00
Bill Hollings
76aca81fac
Merge pull request #1561 from BeastLe9enD/master
Added optimized support for VK_COMMAND_BUFFER_USAGE_ONE_TIME_SUBMIT_BIT when MVK_CONFIG_PREFILL_METAL_COMMAND_BUFFERS is used
2022-04-05 14:12:03 -04:00
BeastLe9enD
1955c161a6
Added optimized support for VK_COMMAND_BUFFER_USAGE_ONE_TIME_SUBMIT_BIT when MVK_CONFIG_PREFILL_METAL_COMMAND_BUFFERS is used 2022-04-05 18:01:52 +02:00
Bill Hollings
32f5bee40d
Merge pull request #1557 from billhollings/vulkan-extensions-os-version-support
Use MVKExtensions.def to check OS version support for all Vulkan extensions.
2022-04-03 22:11:27 -04:00
Bill Hollings
d84f6b69cf Use MVKExtensions.def to check OS version support for all Vulkan extensions.
In the interests of Single Source of Truth, OS version support is now populated
in MVKExtensions.def, and that info is used to validate each Vulkan extension
against OS version support for the functionality required by the extension, with
the default being unsupported, unless otherwise indicated in MVKExtensions.def.

- Add OS version info for each extension in MVKExtensions.def.
- mvkIsSupportedOnPlatform() checks every extension for OS version support,
  not just a separately-populated list of OS version limitations (that defaulted
  to supported, instead of unsupported).
- Visually clean up MVKExtensions.def for easier reading.
2022-04-03 20:31:55 -04:00
Bill Hollings
6e8de7c696
Merge pull request #1550 from billhollings/remove-std-iterator
Remove use of C++17 deprecated feature: std::iterator.
2022-03-23 10:25:23 -04:00
Bill Hollings
0bb6917979 Remove use of C++17 deprecated feature: std::iterator.
std::iterator is deprecated in C++17, which triggers multiple compilation warnings.
Update MVKSmallVector::iterator to explicitly specify iterator traits,
instead of subclassing from std::iterator.
Qualify use of std::remove() in mvkRemoveAllOccurances(),
to eliminate resolution ambiguity.
2022-03-22 10:03:56 -04:00
Bill Hollings
7df32cdc06
Merge pull request #1549 from billhollings/xcode-update
Update Xcode build settings validation, and GitHub CI settings.
2022-03-21 22:53:29 -04:00
Bill Hollings
6011bfe05d Update Xcode build settings validation, and GitHub CI settings.
Update to Xcode 13.3 build settings validation.
Update GitHub CI settings to latest macOS and Xcode 13.2.1.
2022-03-21 20:31:33 -04:00
Bill Hollings
fada8e08c7
Merge pull request #1547 from billhollings/pipeline-layout-compatibility-fixes
Fixes to pipeline layout compatibility
2022-03-16 14:05:22 -04:00
Bill Hollings
1b6b8bc992 Move implicit buffers to top of Metal buffer index range.
To better support pipeline layout compatibility between pipelines with differing
quantities of descriptor sets, move the buffer indexes used by implicit buffers to
the top end of the Metal buffer index range, below vertex and tessellation buffers.

MVKPipeline calculates implicit buffer indexes based on vertex and tessellation
buffers required by pipeline, instead of based on descriptors in MVKPipelineLayout.

MVKPipeline track buffer index counts consumed by MVKPipelineLayout, to validate
room for implicit buffers.
2022-03-15 11:52:15 -04:00
Bill Hollings
2a17f757e1 Fixes to pipeline layout compatibility.
For pipeline layout compatibility, consume the Metal resource indexes in this order:
- Consume a fixed number of Metal buffer indexes for Metal argument buffers,
  but only consume them if Metal argument buffers are not being used.
- Consume push constants  Metal buffer index before descriptor set resources,
  but only consume it if the stage uses push constants.
- Consume descriptor set bindings.

In MVKPipelineLayout, separate tracking resource counts from push constants
indexes, and move push constant indexes ahead of descriptor bindings.
In MVKPipeline, track which stages use push constants.
Remove unused and obsolete function declaration in MVKDescriptorSet.h.
2022-03-14 16:29:22 -04:00
Bill Hollings
9cfc946a7c
Merge pull request #1546 from billhollings/retained-resource-object-fixes
Fixes to optimize resource objects retained by descriptors beyond their lifetimes.
2022-03-13 17:05:30 -04:00
Bill Hollings
9f4243ff91 Fixes to optimize resource objects retained by descriptors beyond their lifetimes.
For a resource object that can be retained by descriptors beyond its lifetime,
release memory resources when the object is destroyed by the app. This includes
objects of type MVKBuffer, MVKBufferView, MVKImageView, and MVKSampler.

When the app destroys an MVKBuffer, also detach from the MVKDeviceMemory,
to fix a potential race condition when the app updates the descriptor on
one thread while also freeing the MVKDeviceMemory on another thread.

MVKImageView guard against detached planes while in descriptor.

Add comment to clarify how destroy() is called from release().
2022-03-13 14:54:54 -04:00
Bill Hollings
eba6a04a91
Merge pull request #1545 from billhollings/avoid-adjusting-sRGB-clear-colors
Avoid adjusting SRGB clear color values by half-ULP.
2022-03-11 12:51:46 -05:00
Bill Hollings
cf3b5968ef Avoid adjusting SRGB clear color values by half-ULP.
For GPUs that round float clear colors down, a half-ULP adjustment is performed
on normalized formats. But this adjustment should not be performed on SRGB formats,
which Vulkan requires to be treated as linear, with the value managed by the app.
2022-03-11 11:24:05 -05:00
Bill Hollings
cd2146c229
Merge pull request #1544 from billhollings/fix-maxSamplerAllocationCount
Remove limit on VkPhysicalDeviceLimits::maxSamplerAllocationCount when not using Metal argument buffers.
2022-03-10 21:44:01 -05:00
Bill Hollings
a91a3d000a Remove limit on VkPhysicalDeviceLimits::maxSamplerAllocationCount
when not using Metal argument buffers.
2022-03-10 18:12:54 -05:00
Bill Hollings
d3b5d113f7
Merge pull request #1543 from billhollings/fix-gpu-counter-mtlfence-deletion
Fix deletion of GPU counter MTLFence while it is being used by MTLCommandBuffer.
2022-03-10 13:51:34 -05:00
Bill Hollings
2aadca70ce Fix deletion of GPU counter MTLFence while it is being used by MTLCommandBuffer.
Move release of GPU counter MTLFence from MVKCommandEncoder destructor
to MTLCommandBuffer completion handler.
2022-03-10 13:25:09 -05:00
Bill Hollings
da0f15c7b0
Merge pull request #1541 from billhollings/shaderconverter-tool-default-msl-version
MoltenVKShaderConverter tool defaults to the highest MSL version supported on runtime OS.
2022-03-09 13:54:15 -05:00
Bill Hollings
728182cf75 MoltenVKShaderConverter tool defaults to the
highest MSL version supported on runtime OS.

Update to latest SPIRV-Cross.
Update Docs/Whats_New.md document.
2022-03-08 21:22:25 -05:00
Bill Hollings
c5ce77fff3
Merge pull request #1540 from billhollings/reinstate-gpu-memory-barriers
Reinstate memory barriers on non-Apple GPUs.
2022-03-08 20:58:26 -05:00
Bill Hollings
3114e16457 Fix build for Xcode 11.7. 2022-03-08 17:52:50 -05:00
Bill Hollings
6327b767e0 Reinstate memory barriers on non-Apple GPUs.
Ensure non-Apple GPU's enable memory barriers.
A previous commit inadvertently disabled GPU memory barriers.

Change tests for memory barriers to runtime test for Apple GPU, instead of
build-time test for Apple Silicon, to accommodate running on Rosetta2, and
refactor tests for Apple Silicon and OS version on some macOS GPU feature settings.
2022-03-08 17:01:50 -05:00
Bill Hollings
5d7b7eaba6
Merge pull request #1539 from redstrate/master
Enable base vertex instance support in shader conversion
2022-03-08 16:50:57 -05:00
Joshua Goins
b6c392541c
Check if metal device supports base vertex instance drawing
Co-authored-by: Bill Hollings <bill.hollings@brenwill.com>
2022-03-08 14:17:55 -05:00
Joshua Goins
d05537dbe5 Enable base vertex instance support in shader conversion 2022-03-06 17:07:54 -05:00
Bill Hollings
c09bcd0972
Merge pull request #1531 from billhollings/stencil-only-depth-store-action
Don't attempt to store the depth component of a stencil-only renderpass attachment.
2022-02-24 14:37:08 -05:00
Bill Hollings
e42b33e593 Don't attempt to store the depth component of a stencil-only renderpass attachment.
For a combined depth-stencil format in a MVKImageView attachment with
VK_IMAGE_ASPECT_STENCIL_BIT, the attachment format may have been swizzled
to a stencil-only format. In this case, we want to guard against an attempt
to store the non-existent depth component.

Pass MVKImageView attachment to MVKRenderPassAttachment::encodeStoreAction()
and MVKRenderPassAttachment::populateMTLRenderPassAttachmentDescriptor() to
check attachment depth format component.

Consolidate calls to MVKImageView::populateMTLRenderPassAttachmentDescriptor() by calling
it from within MVKRenderPassAttachment::populateMTLRenderPassAttachmentDescriptor().
2022-02-24 11:42:12 -05:00
Bill Hollings
a6f5841b91
Merge pull request #1530 from billhollings/align-shader-io-nested-structs
Align flattened shader inputs to previous stage output structs.
2022-02-22 17:29:41 -05:00
Bill Hollings
e28a16d76b Update MoltenVK version number to 1.1.9.
Adjust Whats_New.md to accommodate earlier
trivial 1.1.8 patch release for SDK 1.3.204.1.
2022-02-22 14:18:55 -05:00
Bill Hollings
afd997ab31 Align flattened shader inputs to previous stage output structs.
When flattening shader inputs for stage_in, which are to be read from a buffer
that was populated as nested structs during an earlier stage, the structs will
be aligned according to C++ rules, which can affect the alignment of the first
member of the flattened input struct.

Add SPIRVShaderOutput::firstStructMemberAlignment to track the alignment
requirements of the first member of a nested structure, and recursively
determine the alignment of the first member of each nested output structure.

Move sizeOfOutput() from MVKPipeline.mm to SPIRVReflection.h,
rename to getShaderOutputSize(), and add getShaderOutputAlignment()
to extract member alignment.
2022-02-22 12:17:15 -05:00
Bill Hollings
24f802a5dd
Merge pull request #1527 from billhollings/update-glslang
Update glslang version, to use python3 in glslang scripts, to replace missing python on macOS 12.3.
2022-02-16 14:48:24 -05:00
Bill Hollings
f7ca132844 Update glslang version, to use python3 in glslang scripts,
to replace missing python on macOS 12.3.
2022-02-16 14:02:01 -05:00
Bill Hollings
2fa625f962
Merge pull request #1523 from billhollings/rmv-api-warn
Remove logged warning if MoltenVK does not support VkApplicationInfo::apiVersion value.
2022-02-09 13:53:44 -05:00
Bill Hollings
16408fd6ae Remove logged warning if MoltenVK does not support VkApplicationInfo::apiVersion value.
Update MoltenVK version to 1.1.8.
Minor spelling fixes in comments.
2022-02-09 13:36:08 -05:00
Bill Hollings
8218606f44
Merge pull request #1522 from billhollings/sdk-1.3.204
Update dependency libraries to match Vulkan SDK 1.3.204.
2022-02-06 20:46:12 -05:00
Bill Hollings
24ff2106d9 Update dependency libraries to match Vulkan SDK 1.3.204.
Update What's New document.
2022-02-06 19:55:44 -05:00
Bill Hollings
24c2315072
Merge pull request #1520 from billhollings/simulator-gpu-counter-sets
Fix use of GPU counter sets on older versions of iOS running on the simulator.
2022-02-05 13:23:26 -05:00
Bill Hollings
13a0f6abee Fix use of GPU counter sets on older versions of iOS running on the simulator.
Replace test for MVK_APPLE_SILICON with test for MVK_MACOS plus Apple1 GPU.
2022-02-05 12:23:05 -05:00
Bill Hollings
101ea9eec1
Merge pull request #1519 from cfnptr/master
Update license year
2022-02-04 11:27:57 -05:00
Nikita Fediuchin
be51560ba9 Update license year 2 2022-02-04 17:36:53 +02:00
Nikita Fediuchin
4efb90b3c1 Update license year 2022-02-04 13:33:27 +02:00