This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Rename AMDGPUCFGStructurizer to R600MachineCFGStructurizer
ClosedPublic

Authored by foad on Feb 18 2022, 6:25 AM.

Details

Summary

Previously the name of the class (AMDGPUCFGStructurizer) did not
match the name of the file (AMDILCFGStructurizer).

Standardize on the name R600MachineCFGStructurizer by analogy with
AMDGPUMachineCFGStructurizer.

Diff Detail

Event Timeline

foad created this revision.Feb 18 2022, 6:25 AM
foad requested review of this revision.Feb 18 2022, 6:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 18 2022, 6:25 AM
foad added a comment.Feb 18 2022, 6:26 AM

The motivation for this is to get rid of the anomalous AMDIL prefix on the filename, which I don't think has any meaning here, and it annoys me every time I try to tab-complete:

$ ls lib/Target/AMDGPU/AMD<TAB>
arsenm accepted this revision.Feb 18 2022, 7:06 AM
This revision is now accepted and ready to land.Feb 18 2022, 7:06 AM
This revision was landed with ongoing or failed builds.Feb 18 2022, 7:11 AM
This revision was automatically updated to reflect the committed changes.