LLVM Bugzilla is read-only and represents the historical archive of all LLVM issues filled before November 26, 2021. Use github to submit LLVM bugs

Bug 36178 - Merge r323908 into the 6.0 branch : AMDGPU: Add intrinsics llvm.amdgcn.cvt.{pknorm.i16, pknorm.u16, pk.i16, pk.u16}
Summary: Merge r323908 into the 6.0 branch : AMDGPU: Add intrinsics llvm.amdgcn.cvt.{p...
Status: RESOLVED FIXED
Alias: None
Product: new-bugs
Classification: Unclassified
Component: new bugs (show other bugs)
Version: 6.0
Hardware: All All
: P enhancement
Assignee: Unassigned LLVM Bugs
URL:
Keywords:
Depends on:
Blocks: release-6.0
  Show dependency tree
 
Reported: 2018-01-31 13:36 PST by Marek Olšák
Modified: 2018-02-02 08:24 PST (History)
3 users (show)

See Also:
Fixed By Commit(s): r323908

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marek Olšák 2018-01-31 13:36:57 PST 
Is it OK to merge the following revision(s) to the 6.0 branch?
Comment 1 Marek Olšák 2018-01-31 13:37:02 PST 
https://reviews.llvm.org/rL323908
Comment 2 Hans Wennborg 2018-02-02 05:52:52 PST 
This looks like new functionality; why does it need merging?

Matt, I'll let you call it.
Comment 3 Marek Olšák 2018-02-02 06:12:38 PST 
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.
Comment 4 Matt Arsenault 2018-02-02 08:17:18 PST 
I think there's low risk from adding new intrinsics, so this seems fine
Comment 5 Hans Wennborg 2018-02-02 08:24:20 PST 
(In reply to Matt Arsenault from comment #4)
> I think there's low risk from adding new intrinsics, so this seems fine

Okay, r324103.