Remove assert for memory alignment

[lolzballs]

This assert is not useful -- the size returned by the implementation is not supposed to be aligned. The alignment is for sub-allocating resources within a device memory allocation. In addition, allocations of memory from vkAllocateMemory are guaranteed to be aligned to the largest memory alignment requirement.

Fixes #51.

[dabrain34]

can you please @zlatinski give your feedback on this assert ? what was the initial idea on having it ?

[dabrain34]

Can you backport this patch from NVPro as it is more complete ?

[lolzballs]

Can you backport this patch from NVPro as it is more complete ?

@zlatinski, I'm not sure why allocationSize needs to be aligned? The only VU that indicates allocationSize needs to be aligned is VUID-VkMemoryAllocateInfo-allocationSize-01745, which doesn't apply to us because we are not importing.

[zlatinski]

Please see 1a45c233e5461a39a2b09189d62f12a2370e313f from NVPRO samples.
vulkan: fix memory allocation size alignment in CreateDeviceMemory

[lolzballs]

Please see 1a45c233e5461a39a2b09189d62f12a2370e313f from NVPRO samples. vulkan: fix memory allocation size alignment in CreateDeviceMemory

@zlatinski, in that change you align allocationSize, but there is no requirement that it be aligned. Can you please double check why you added it in NVPRO?

[zlatinski]

in that change you align allocationSize, but there is no requirement that it be aligned. Can you please double check why you added it in NVPRO?

This is per the API specifying the alignment within the memoryRequirements.alignment, which is usually coming from vkGetBufferMemoryRequirements(device, buffer, &memoryRequirements).

[lolzballs]

in that change you align allocationSize, but there is no requirement that it be aligned. Can you please double check why you added it in NVPRO?

This is per the API specifying the alignment within the memoryRequirements.alignment, which is usually coming from vkGetBufferMemoryRequirements(device, buffer, &memoryRequirements).

The VkMemoryRequirements::alignment refers to the the alignment within an allocation for the buffer. i.e. if you allocate a big chunk of memory and you want to suballocate within it, each buffer must be at an offset that is aligned to VkMemoryRequirements::alignment.

AFAIK (and confirmed with validation layers) there is no requirement for the allocation size to the aligned to VkMemoryRequirements::alignment.