Skip to content

Removal of Vulkan API support.#648

Merged
coldav merged 1 commit into
uxlfoundation:mainfrom
coldav:colin/remove_vulkan_sdk_reference
May 16, 2025
Merged

Removal of Vulkan API support.#648
coldav merged 1 commit into
uxlfoundation:mainfrom
coldav:colin/remove_vulkan_sdk_reference

Conversation

@coldav
Copy link
Copy Markdown
Collaborator

@coldav coldav commented Jan 17, 2025

Overview

Removal of Vulkan API support

Reason for change

Agreed to remove it to reduce overheads.

Description of change

This removes Vulkan API support from the construction kit, including
all testing and documentation - see
https://github.com/uxlfoundation/oneapi-construction-kit/blob/rfcs/rfc-0003.md
for more information.

This also increases the version to 5.0.0. Note that some code which is
only being used by Vulkan remains after this commit. This will be reviewed
in a later commit.

@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch 5 times, most recently from ba45728 to 2926f66 Compare May 8, 2025 09:15
@coldav coldav changed the title DRAFT: Reference removal of Vulkan SDK dependency Removal of Vulkan API support, update to version 5.0.0 May 8, 2025
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch 2 times, most recently from a636eac to 5495342 Compare May 8, 2025 10:49
@hvdijk
Copy link
Copy Markdown
Collaborator

hvdijk commented May 8, 2025

This also increases the version to 5.0.0.

Why is this part of this PR?

@coldav
Copy link
Copy Markdown
Collaborator Author

coldav commented May 8, 2025

This also increases the version to 5.0.0.

Why is this part of this PR?

I thought as it was removing an API, this counted as a breaking change and we had to up the version. If you disagree we can remove it and do that later.

@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch 2 times, most recently from 1d269bc to 6bd694c Compare May 8, 2025 13:58
Comment thread doc/cmake.rst Outdated
Comment thread doc/cmake.rst Outdated
Comment thread doc/cmake.rst Outdated
Comment thread doc/developer-guide.md Outdated
Comment thread doc/getting-started.rst Outdated
Comment thread doc/overview/introduction/architecture.rst Outdated
Comment thread doc/overview/introduction/architecture.rst Outdated
Comment thread doc/overview/preface.rst Outdated
Comment thread modules/compiler/compiler_pipeline/source/pass_functions.cpp Outdated
Comment thread modules/compiler/include/compiler/module.h
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch from 6bd694c to c04634c Compare May 9, 2025 09:11
@coldav coldav changed the title Removal of Vulkan API support, update to version 5.0.0 Removal of Vulkan API support. May 13, 2025
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch from c04634c to 7e47cbd Compare May 13, 2025 09:19
Comment thread doc/cmake.rst Outdated
Comment thread doc/developer-guide.md Outdated
Comment thread doc/getting-started.rst Outdated
Comment thread doc/modules/mux.rst Outdated
Comment thread doc/overview/introduction/compiler-view.rst Outdated
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch 2 times, most recently from 8166e0a to fb6e0b8 Compare May 15, 2025 14:40
Comment thread doc/cmake.rst Outdated
Comment thread doc/developer-guide.md Outdated
Comment thread doc/getting-started.rst Outdated
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch from fb6e0b8 to 5380eb5 Compare May 15, 2025 16:20
Comment thread doc/developer-guide.md Outdated
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch from 5380eb5 to cd08d28 Compare May 15, 2025 16:32
This removes Vulkan API support from the construction kit, including
all testing and documentation - see
https://github.com/uxlfoundation/oneapi-construction-kit/blob/rfcs/rfc-0003.md
for more information.

Note that some code which is only being used by Vulkan remains after this
commit. This will be reviewed in a later commit.

Note that the next release should increase the version to 5.0.0 after
this breaking change.
@coldav coldav force-pushed the colin/remove_vulkan_sdk_reference branch from cd08d28 to a638e42 Compare May 15, 2025 16:37
@coldav coldav merged commit df8de76 into uxlfoundation:main May 16, 2025
14 checks passed
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 21, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 22, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 26, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 27, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 27, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 27, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 27, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 27, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Aug 29, 2025
…sdk_reference

Removal of Vulkan API support.
coldav pushed a commit to coldav/oneapi-construction-kit that referenced this pull request Sep 5, 2025
…sdk_reference

Removal of Vulkan API support.
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