This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] IsFlatScratch/Global -> FlatScratch/Global
ClosedPublic

Authored by sebastian-ne on Apr 8 2021, 7:16 AM.

Details

Summary

Remove 'Is' from IsFlatScratch/Global. NFC

Should I do the same for is_flat_scratch/global?

Diff Detail

Event Timeline

sebastian-ne created this revision.Apr 8 2021, 7:16 AM
sebastian-ne requested review of this revision.Apr 8 2021, 7:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2021, 7:16 AM
foad added a comment.Apr 8 2021, 7:47 AM

Should I do the same for is_flat_scratch/global?

I don't think it matters, since it is only used in FLATInstructions.td.

foad accepted this revision.Apr 8 2021, 7:49 AM

Seems fine to me. There is not much consistency in the naming of the SIInstrFlags flags.

This revision is now accepted and ready to land.Apr 8 2021, 7:49 AM
foad added a subscriber: rampitec.

... but adding @rampitec for awareness.

This revision was automatically updated to reflect the committed changes.