Skip to content

Conversation

rbegam
Copy link
Contributor

@rbegam rbegam commented Jul 31, 2020

Signed-off-by: rbegam [email protected]

@rbegam
Copy link
Contributor Author

rbegam commented Jul 31, 2020

please approve @bader

@bader
Copy link
Contributor

bader commented Jul 31, 2020

please approve @bader

I can do this, but in order to merge this PR code owners must approve as well. In this particular case runtime code owners must approve.

@rbegam
Copy link
Contributor Author

rbegam commented Jul 31, 2020

ping @alexbatashev

@bader
Copy link
Contributor

bader commented Aug 3, 2020

@rbegam, please, fix regressions in pre-commit testing.

@bader
Copy link
Contributor

bader commented Aug 14, 2020

@rbegam, ping.

@romanovvlad
Copy link
Contributor

Closing due to lack of activity. Please, feel free to reopen.

@@ -1,4 +1,4 @@
// REQUIRES: opencl || level0
Copy link
Contributor

Choose a reason for hiding this comment

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

Please, remove "|| level0".

@rbegam rbegam reopened this Aug 31, 2020
@rbegam
Copy link
Contributor Author

rbegam commented Aug 31, 2020

I am submitting a new PR related to this lit-test.

@rbegam rbegam closed this Aug 31, 2020
jsji pushed a commit that referenced this pull request Nov 30, 2023
There is a fundamental misalignment between when translating llvm.ptr.annotation to SPIRV's OpMemberDecorate.

llvm.ptr.annotation is applied onto a pointer.

OpMemberDecorate is applied on a type, which means that all variables of this type will have this Decoration.

This change fixes this misalignment by translating llvm.ptr.annotation to SPIRV's OpDecorate instead, which instead places the decoration onto the pointer.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@3770469
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