Skip to content

Add vulkan renderer API from suijingfeng/vkQuake3 - #841

Closed
llbraughler wants to merge 11 commits into
ioquake:mainfrom
llbraughler:vulkan
Closed

Add vulkan renderer API from suijingfeng/vkQuake3#841
llbraughler wants to merge 11 commits into
ioquake:mainfrom
llbraughler:vulkan

Conversation

@llbraughler

@llbraughler llbraughler commented Nov 23, 2025

Copy link
Copy Markdown

Fixes #812

@llbraughler

Copy link
Copy Markdown
Author

The SDL build path needs cleaned up and a few other tweaks for the other 3 platforms.

@llbraughler

Copy link
Copy Markdown
Author

I can't see why the OpenBSD process is failing. Maybe someone else can help with that.

Lojjik Braughler added 2 commits November 23, 2025 18:43
@NuclearMonster

Copy link
Copy Markdown
Member

re-running the bsd one

@llbraughler

Copy link
Copy Markdown
Author

Ok this is ready for others to test. Before merging, it would be good to squash the commits down and maybe we can leave opengl2 as the default renderer until this has proven reliable on all platforms.

@NuclearMonster

NuclearMonster commented Nov 24, 2025

Copy link
Copy Markdown
Member

I don't believe Vulkan is at all appropriate for defaulting due to availability and stability, unless something has changed? I believe on Mac you still have to use a metal translation layer and on other platforms I've had unpredictable behavior even on modern nvidia.

@NuclearMonster

Copy link
Copy Markdown
Member

Was some copilot or other LLM assistance used to develop this PR?

@llbraughler

Copy link
Copy Markdown
Author

I agree it’s not ready to be the default yet. Right now it’s just helpful to get more eyes on it and let people start using it, while gl2 stays the stable baseline. Longer term, I do expect the classic GL path to get harder to maintain as vendors lean more heavily into modern APIs, so at some point in the future it might make sense to default to Vulkan.

@llbraughler

Copy link
Copy Markdown
Author

Squashing commits

@NuclearMonster

Copy link
Copy Markdown
Member

Was some copilot or other LLM assistance used to develop this PR?

Please answer the question.

@llbraughler

Copy link
Copy Markdown
Author

Yes, I used Copilot for the numerous build errors that prevented compilation

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.

Vulkan Renderer

2 participants