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

Merge r323908 into the 6.0 branch : AMDGPU: Add intrinsics llvm.amdgcn.cvt.{pknorm.i16, pknorm.u16, pk.i16, pk.u16} #35526

Closed
llvmbot opened this issue Jan 31, 2018 · 5 comments
Labels
bugzilla Issues migrated from bugzilla

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 31, 2018

Bugzilla Link 36178
Resolution FIXED
Resolved on Feb 02, 2018 08:24
Version 6.0
OS All
Blocks #35152
Reporter LLVM Bugzilla Contributor
CC @zmodem,@arsenm
Fixed by commit(s) r323908

Extended Description

Is it OK to merge the following revision(s) to the 6.0 branch?

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jan 31, 2018

@zmodem
Copy link
Collaborator

zmodem commented Feb 2, 2018

This looks like new functionality; why does it need merging?

Matt, I'll let you call it.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Feb 2, 2018

There is no effect on existing applications. The new functionality is added so that GPU drivers can start using it since LLVM 6.0 instead of 7.0. The motivation is GPU performance.

@arsenm
Copy link
Contributor

arsenm commented Feb 2, 2018

I think there's low risk from adding new intrinsics, so this seems fine

@zmodem
Copy link
Collaborator

zmodem commented Feb 2, 2018

I think there's low risk from adding new intrinsics, so this seems fine

Okay, r324103.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Development

No branches or pull requests

3 participants