This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add TransVALU to GFX11 scheduling model
ClosedPublic

Authored by foad on Aug 11 2023, 5:49 AM.

Details

Summary

The GFX11 scheduling model should be identical to GFX10, except for a
few tweaks to latencies.

This GFX10 change from D100123 was missed when GFX11 was added.

Diff Detail

Event Timeline

foad created this revision.Aug 11 2023, 5:49 AM
Herald added a project: Restricted Project. · View Herald Transcript
foad requested review of this revision.Aug 11 2023, 5:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 5:49 AM
critson accepted this revision.Aug 13 2023, 6:56 PM
critson added a subscriber: critson.

LGTM

Scheduling change does not cause any other test changes?

This revision is now accepted and ready to land.Aug 13 2023, 6:56 PM
foad added a comment.Aug 14 2023, 3:17 AM

Scheduling change does not cause any other test changes?

Yeah, like the GFX10 patch said, "This doesn't seem to have any effect". I'm doing this for consistency and as a basis for more patches to hopefully enable trans coexecution.

This revision was automatically updated to reflect the committed changes.