Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D50626

AMDGPU: Implement llvm.amdgcn.icmp/fcmp for i16/f16
ClosedPublic
Actions

Authored by arsenm on Aug 13 2018, 3:00 AM.

Details

Reviewers
rampitec
scchan
Summary

Also support these on targets without support for these,
since it will allow us to freely create thes in instcombine.

Diff Detail

Event Timeline

arsenm created this revision.Aug 13 2018, 3:00 AM
Herald added subscribers: t-tye, tpr, dstuttard and 4 others. · View Herald TranscriptAug 13 2018, 3:00 AM
arsenm added a parent revision: D50625: DAG: Try to custom lower when promoting float operands.Aug 13 2018, 3:00 AM
rampitec accepted this revision.Aug 13 2018, 3:02 AM

LGTM

This revision is now accepted and ready to land.Aug 13 2018, 3:02 AM
scchan accepted this revision.Aug 13 2018, 7:44 AM
scchan added a subscriber: scchan.

LGTM

arsenm closed this revision.Aug 15 2018, 2:26 PM

r339819

Revision Contents
Changeset List

PathSize
lib/
Target/
AMDGPU/
93 lines
26 lines
test/
CodeGen/
AMDGPU/
190 lines
160 lines

Diff 160315

View Options

lib/Target/AMDGPU/SIISelLowering.cpp

Loading...
View Options

lib/Target/AMDGPU/VOPCInstructions.td

Loading...
View Options

test/CodeGen/AMDGPU/llvm.amdgcn.fcmp.ll

Loading...
View Options

test/CodeGen/AMDGPU/llvm.amdgcn.icmp.ll

Loading...