This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Introduce AMDGPU::SGPR_SPILL asm comment flag
ClosedPublic

Authored by yassingh on Jun 26 2023, 4:49 AM.

Details

Summary

Use this flag to give more context to implicit def comments
in assembly.

Diff Detail

Event Timeline

yassingh created this revision.Jun 26 2023, 4:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2023, 4:49 AM
yassingh requested review of this revision.Jun 26 2023, 4:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2023, 4:49 AM
arsenm accepted this revision.Jun 26 2023, 5:09 AM
arsenm added inline comments.
llvm/lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp
254

getSubtarget<GCNSubtarget>()?

llvm/lib/Target/AMDGPU/AMDGPUAsmPrinter.h
119

I thought the mechanism for this was much more general but OK

This revision is now accepted and ready to land.Jun 26 2023, 5:09 AM
yassingh updated this revision to Diff 557220.Sep 21 2023, 11:51 PM

Rebase before merge

yassingh closed this revision.Sep 28 2023, 10:46 PM