Page MenuHomePhabricator

AMDGPU: Use default operand for VOP3P clamp
ClosedPublic

Authored by arsenm on Feb 19 2020, 6:18 AM.

Details

Reviewers
rampitec
Summary

We don't use this, and matching from the def doesn't make much sense.

There are multiple tablegen bugs with default operand
handling. undef_tied_input should work to handle the vdst_in
correctly, but this breaks the operand register class constraint which
it should be able to infer.

Diff Detail

Event Timeline

arsenm created this revision.Feb 19 2020, 6:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 19 2020, 6:18 AM
rampitec accepted this revision.Feb 19 2020, 9:31 AM
rampitec added inline comments.
llvm/lib/Target/AMDGPU/VOP3PInstructions.td
34

Typo: thiss

This revision is now accepted and ready to land.Feb 19 2020, 9:31 AM