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

[Backport to 8] Support SPV_EXT_relaxed_printf_string_address_space extension #1832

Merged
merged 6 commits into from
Feb 8, 2023

Conversation

vmaksimo
Copy link
Contributor

@vmaksimo vmaksimo commented Feb 3, 2023

This backport is a squash of #1364, #1749, #1818

Also 4 changes for SPIR-V friendly IR were backported to allow SPIR-V friendly mangling.
Without these patches the extension can't be used, because llvm-spirv can't generate printf function for different address spaces arguments in reverse translation.

Removed extra variables, simplified code, removed dead code related to
"read_image" handling: it is not present in OpenCL.std extended instruction
set and therefore we can't encounter it in `transOCLBuiltinFromExtInst`
function.
Copy link
Contributor

@MrSidims MrSidims left a comment

Choose a reason for hiding this comment

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

LGTM
@vmaksimo please squash last three commits into one called "Adjust SPV_EXT_relaxed_printf_string_address_space for 8.0 branch" or something like this.

AlexeySachkov and others added 5 commits February 8, 2023 03:33
Added possibility to emit SPIR-V friendly IR instead of OpenCL-C-like
mangled function names for instructions from OpenCL extended instruction
set.
…CL pass

This change also fixes vstore translation for SPV-IR generation path and adds
test for vstore and printf.
@MrSidims MrSidims merged commit 09dd9ac into KhronosGroup:llvm_release_80 Feb 8, 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.

4 participants