Fix: Out-of-boounds memory type index allocation for discrete GPUs

[SpinnerX]

Issue Overview

The way vulkan-cpp allows for specifying for memory allocations is managed by the vk::physical_device to retrieve the bit masks of the available memory properties. The physical device to determine the capabilities themask to determine where resources should live in.

On integrated physical device types, these masks overlap. Whereas on discrete physical device types have a varied different memory type and heap configurations.

Which caused cases where the buffers memory requirements and the requested device property mask had no common bits.

• Resulting in the bitmasking to be 0.
• Passing 0 to std::countr_zero caused to return 32.
• Passing an invalid index type of 32 to the Vulkan vkAllocateMemory APIs.

Fixes

To fix this, I added a fallback to handle such edge case for specifying std::countr_zero and passing in the memoryTypeBits provided by the buffer memory requirements.

The fallback will evaluate the supported memory indexes and choose the first bit that represents the available supported memory index for allocation for the specific device type.

[SpinnerX]

Merging this PR.