Skip to content

Fix loader include under V3 - #33

Merged
antono3 merged 2 commits into
masterfrom
fix/v3-loader-include
Sep 12, 2026
Merged

Fix loader include under V3#33
antono3 merged 2 commits into
masterfrom
fix/v3-loader-include

Conversation

@antono3

@antono3 antono3 commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • resolve the private Volk loader bridge through @VMODROOT so generated C does not depend on the compiler's working directory
  • add a pinned, authoritative V3 compile-and-test matrix entry using -new-compiler
  • retain the live V master compatibility lane as an advisory drift check

Why

v_vulkan_video exposed this under V3: generated src.c included loader_bridge.h without an include path and GCC could not find it. The released compiler happened to preserve the module-relative lookup. Using V's documented module-root path makes the binding portable across both frontends.

Local verification

  • V 0.5.2 generator compile smoke with GCC
  • V 0.5.2 ergonomic tests: 4/4 passed
  • workflow YAML parse and git diff --check

The PR matrix provides the authoritative pinned V3 regression check.

@antono3
antono3 merged commit 79de570 into master Sep 12, 2026
8 checks passed
@antono3
antono3 deleted the fix/v3-loader-include branch September 12, 2026 12: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