This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add MC layer support for v_fmac_legacy_f32
ClosedPublic

Authored by foad on Oct 12 2020, 8:56 AM.

Details

Summary

This instruction was introduced in GFX10.3, reusing the opcode of
v_mac_legacy_f32 from GFX10.1.

Diff Detail

Event Timeline

foad created this revision.Oct 12 2020, 8:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 12 2020, 8:56 AM
foad requested review of this revision.Oct 12 2020, 8:56 AM
arsenm added inline comments.Oct 12 2020, 9:27 AM
llvm/test/MC/AMDGPU/gfx1030_unsupported.s
1

The sutarget features are broken and should be all lowercase

It also seems to lack changes in SIFoldOperands, SIInstrInfo::FoldImmediate, SIInstrInfo::convertToThreeAddress, SIInstrInfo::canShrink. I think these can go in a separate change.

foad added a comment.Oct 13 2020, 1:05 AM

It also seems to lack changes in SIFoldOperands, SIInstrInfo::FoldImmediate, SIInstrInfo::convertToThreeAddress, SIInstrInfo::canShrink. I think these can go in a separate change.

Right. I'll change the summary to make it clear that this just adds asm/dis support.

llvm/test/MC/AMDGPU/gfx1030_unsupported.s
1

Do you mean they're broken in some other way, as well as not being lowercase?

rampitec added inline comments.Oct 13 2020, 1:13 AM
llvm/test/MC/AMDGPU/gfx1030_unsupported.s
1

Shall be enough just to change the case. Subpar, but on par with the rest of the tests.

foad updated this revision to Diff 297804.Oct 13 2020, 2:56 AM

Lowercase subtarget feature names.

foad retitled this revision from [AMDGPU] Add v_fmac_legacy_f32 to [AMDGPU] Add MC layer support for v_fmac_legacy_f32.Oct 13 2020, 5:41 AM
This revision is now accepted and ready to land.Oct 13 2020, 8:20 AM
This revision was landed with ongoing or failed builds.Oct 13 2020, 2:07 PM
This revision was automatically updated to reflect the committed changes.