This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Make saturating add/sub legal for DAG path
ClosedPublic

Authored by arsenm on Jul 14 2020, 7:55 PM.

Details

Reviewers
rampitec
foad

Diff Detail

Event Timeline

arsenm created this revision.Jul 14 2020, 7:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2020, 7:55 PM
arsenm updated this revision to Diff 278187.Jul 15 2020, 7:43 AM

Fix test in wrong patch

This revision is now accepted and ready to land.Jul 15 2020, 11:18 AM
foad added inline comments.Jul 16 2020, 3:01 AM
llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
576

Can you update the comment? "v2f16" is not relevant to saturated arithmetic.

llvm/lib/Target/AMDGPU/SIISelLowering.cpp