This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Eliminate SIMCCodeEmitter and de-virtualise encoding methods.
ClosedPublic

Authored by kosarev on Jul 3 2023, 5:19 AM.

Diff Detail

Event Timeline

kosarev created this revision.Jul 3 2023, 5:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 3 2023, 5:19 AM
kosarev requested review of this revision.Jul 3 2023, 5:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 3 2023, 5:19 AM
foad added a comment.Jul 3 2023, 5:30 AM

Looks like a nice cleanup to me but can you please rename SIMCCodeEmitter.cpp to AMDGPUMCCodeEmitter.cpp to match the class name, either in this patch or a separate patch?

Looks like a nice cleanup to me but can you please rename SIMCCodeEmitter.cpp to AMDGPUMCCodeEmitter.cpp to match the class name, either in this patch or a separate patch?

Sure, will do with a separate patch.

arsenm accepted this revision.Jul 4 2023, 6:58 AM
This revision is now accepted and ready to land.Jul 4 2023, 6:58 AM
This revision was landed with ongoing or failed builds.Jul 5 2023, 2:13 AM
This revision was automatically updated to reflect the committed changes.
llvm/lib/Target/AMDGPU/MCA/AMDGPUCustomBehaviour.cpp