Skip to content

Use fixed-width C int for Vulkan FD exports - #58

Merged
antono3 merged 1 commit into
masterfrom
fix/v3-fd-c-int32
Sep 23, 2026
Merged

antono3 merged 1 commit into
masterfrom
fix/v3-fd-c-int32

Conversation

@antono3

@antono3 antono3 commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Use i32 for the Vulkan registry's C int fields and parameters. In particular, vkGetMemoryFdKHR, vkGetSemaphoreFdKHR, and vkGetFenceFdKHR write to 32-bit C int* outputs. V3's 64-bit int leaves the upper half of an initialized FD unchanged, so callers can receive a negative value even when Vulkan reports success.

This source reflects the generator mapping in antono2/v_vulkan_bindings#16. The generated C.int placeholder is no longer needed.

Verification

The particle call-site update is a separate antono2/opencl PR.

@antono3
antono3 merged commit c437eac into master Sep 23, 2026
8 checks passed
@antono3
antono3 deleted the fix/v3-fd-c-int32 branch September 23, 2026 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants