Skip to content

Conversation

@M2-TE
Copy link
Contributor

@M2-TE M2-TE commented Jan 14, 2026

Using VULKAN_HPP_PRECOMPILE did not work before, as the PCH and samples sometimes had conflicting VULKAN_HPP_DISPATCH_LOADER_DYNAMIC values. Changes made to solve this:

  1. Samples simply use include directories instead of linking to Vulkan::Hpp. It would otherwise need some kind of "completely clean" Vulkan target with no compile definitions.
  2. Added new utils library variants utils_static and utils_dynamic for the respective samples.
  3. Extended the PCH of samples with more headers. Reused by all variants (static, dynamic, RAII).
  4. Enabled VULKAN_HPP_PRECOMPILE on CI

These changes quite noticably reduced the compile times of the sample build stage.

@sharadhr
Copy link
Contributor

sharadhr commented Jan 14, 2026

To further work on this, my goal after #2358 is to build the samples with the Vulkan module. It would be a further stress/compile test to see how robust the module is.

Copy link
Contributor

@asuessenbach asuessenbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice approach. Simple, but efficient.

Just one issue:

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.

3 participants