Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SPV_EXT_relaxed_printf_string_address_space extension #1749

Conversation

MrSidims
Copy link
Contributor

@MrSidims MrSidims commented Nov 30, 2022

The extension was added in KhronosGroup/SPIRV-Registry#148

This extension relaxes the address space requirement of the printf builtin
to support the cl_ext_relaxed_printf_string_address_space OpenCL C extension.

Starting from this PR SPV_INTEL_non_constant_addrspace_printf will be step by step deprecated.

Signed-off-by: Sidorov, Dmitry [email protected]

The extension was added in KhronosGroup/SPIRV-Registry#148

Starting from this PR SPV_INTEL_non_constant_addrspace_printf will
be step by step deprecated.

Signed-off-by: Sidorov, Dmitry <[email protected]>
Signed-off-by: Sidorov, Dmitry <[email protected]>
@MrSidims MrSidims merged commit a9391f7 into KhronosGroup:main Dec 2, 2022
vmaksimo pushed a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 19, 2023
…nsion

The original change: KhronosGroup#1749

The extension was added in KhronosGroup/SPIRV-Registry#148

Starting from this PR SPV_INTEL_non_constant_addrspace_printf will
be step by step deprecated.

Signed-off-by: Sidorov, Dmitry <[email protected]>
vmaksimo pushed a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 19, 2023
The original change KhronosGroup#1749

The extension was added in KhronosGroup/SPIRV-Registry#148

Starting from this PR SPV_INTEL_non_constant_addrspace_printf will
be step by step deprecated.

Signed-off-by: Sidorov, Dmitry <[email protected]>
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 20, 2023
This change continues KhronosGroup#1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.
MrSidims pushed a commit that referenced this pull request Jan 21, 2023
This change continues #1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 23, 2023
…roup#1818)

This change continues KhronosGroup#1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.

Signed-off-by: Maksimova, Viktoria <[email protected]>
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 23, 2023
…roup#1818)

This change continues KhronosGroup#1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.

Signed-off-by: Maksimova, Viktoria <[email protected]>
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 23, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 23, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 23, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 23, 2023
MrSidims added a commit that referenced this pull request Jan 24, 2023
…nsion

The original change: #1749

The extension was added in KhronosGroup/SPIRV-Registry#148

Starting from this PR SPV_INTEL_non_constant_addrspace_printf will
be step by step deprecated.

Signed-off-by: Sidorov, Dmitry <[email protected]>
MrSidims pushed a commit that referenced this pull request Jan 24, 2023
This change continues #1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.

Signed-off-by: Maksimova, Viktoria <[email protected]>
MrSidims pushed a commit that referenced this pull request Jan 24, 2023
MrSidims pushed a commit that referenced this pull request Jan 24, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Jan 24, 2023
…roup#1818)

This change continues KhronosGroup#1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.

Signed-off-by: Maksimova, Viktoria <[email protected]>
MrSidims added a commit that referenced this pull request Jan 26, 2023
The original change #1749

The extension was added in KhronosGroup/SPIRV-Registry#148

Starting from this PR SPV_INTEL_non_constant_addrspace_printf will
be step by step deprecated.

Signed-off-by: Sidorov, Dmitry <[email protected]>
MrSidims pushed a commit that referenced this pull request Jan 26, 2023
This change continues #1749.
We are removing SPV_INTEL_non_constant_addrspace_printf extension in
favor of SPV_EXT_relaxed_printf_string_address_space, which are
basically the same.

Signed-off-by: Maksimova, Viktoria <[email protected]>
MrSidims pushed a commit that referenced this pull request Jan 26, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Feb 3, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Feb 6, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Feb 8, 2023
MrSidims pushed a commit that referenced this pull request Feb 8, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Mar 3, 2023
vmaksimo added a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this pull request Mar 3, 2023
MrSidims pushed a commit that referenced this pull request Mar 6, 2023
MrSidims pushed a commit that referenced this pull request Mar 6, 2023
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