Skip to content

Conversation

@Fznamznon
Copy link
Contributor

Before structs decomposition USM pointers that are wrapped by structs
were passsed with generic address space. It caused performance issues.
After structs decomposition all pointers are passed with global address
space, this test is added to save this behaviour.

Before structs decomposition USM pointers that are wrapped by structs
were passsed with generic address space. It caused performance issues.
After structs decomposition all pointers are passed with global address
space, this test is added to save this behaviour.
Copy link
Contributor

@premanandrao premanandrao left a comment

Choose a reason for hiding this comment

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

LGTM

@bader bader merged commit 449308d into intel:sycl Aug 11, 2020
jsji pushed a commit that referenced this pull request Jan 25, 2024
Partially load SPIR-V from the stream and decode only selected for the report instructions, needed to retrieve general information about the module: capabilities, extensions, version, memory model and addressing model.

In addition to immediately helpful for back-ends lists of capabilities and extensions declared in SPIR-V module, a general intent also is to extend report details in future by feedbacks about further potentially useful analysis, statistics, etc.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@918036c
jsji pushed a commit that referenced this pull request Feb 1, 2024
The goal of the PR is to add API to SPIR-V LLVM Translator to query error message by an error code as discussed in #2298

A need and possible application is a way to generate human-readable error info by error codes returned by other SPIRV Translator API calls, including getSpirvReport().

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@afe1971
Chenyang-L pushed a commit that referenced this pull request Feb 18, 2025
Improve CUDA/HIP local argument handling
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