This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Remove deprecated V_MUL_LO_I32 from GFX10
ClosedPublic

Authored by Joe_Nash on Jan 4 2021, 10:53 AM.

Details

Summary

It was removed in GFX10 GPUs, but LLVM could
generate it.

Diff Detail

Event Timeline

Joe_Nash created this revision.Jan 4 2021, 10:53 AM
Joe_Nash requested review of this revision.Jan 4 2021, 10:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 4 2021, 10:53 AM
arsenm accepted this revision.Jan 4 2021, 10:55 AM
This revision is now accepted and ready to land.Jan 4 2021, 10:55 AM

Generally speaking it is deprecated but it is supported. Well, not in gfx1030.

Should the disassembler still understand it?

Should the disassembler still understand it?

I think we need to remove it from gfx1030+ only (e.g. !FeatureGFX10_3Insts).

arsenm requested changes to this revision.Jan 4 2021, 11:13 AM
This revision now requires changes to proceed.Jan 4 2021, 11:13 AM
Joe_Nash updated this revision to Diff 314479.Jan 4 2021, 4:18 PM

remove inst from gfx1030 only, instead of all gfx10

arsenm accepted this revision.Jan 4 2021, 4:21 PM
This revision is now accepted and ready to land.Jan 4 2021, 4:21 PM
rampitec requested changes to this revision.Jan 4 2021, 4:22 PM
rampitec added inline comments.
llvm/lib/Target/AMDGPU/AMDGPU.td
1137 ↗(On Diff #314479)

Please use GFX10_3Insts, not encoding.

This revision now requires changes to proceed.Jan 4 2021, 4:22 PM
Joe_Nash updated this revision to Diff 314480.Jan 4 2021, 4:25 PM

changed B_encoding predicate to be 10_3Insts

Joe_Nash marked an inline comment as done.Jan 4 2021, 4:26 PM
rampitec accepted this revision.Jan 4 2021, 4:26 PM

LGTM

This revision is now accepted and ready to land.Jan 4 2021, 4:26 PM
Harbormaster completed remote builds in B83965: Diff 314480.
This revision was automatically updated to reflect the committed changes.