[PATCH] D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging

Venkata Ramanaiah Nalamothu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 16 01:56:23 PST 2021


RamNalamothu updated this revision to Diff 387533.
RamNalamothu added a comment.
Herald added a subscriber: foad.

Rebase on to the ToT.

Taking a relook at the vendor extension range, there are enough non-conflicting encodings available for the DWARF operators being proposed here and also leaves few more encodings available for other future extensions LLVM might see.

Does this look good now to go in?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76878/new/

https://reviews.llvm.org/D76878

Files:
  llvm/docs/AMDGPUDwarfExtensionsForHeterogeneousDebugging.rst
  llvm/docs/AMDGPUUsage.rst
  llvm/include/llvm/BinaryFormat/Dwarf.def
  llvm/include/llvm/MC/MCAsmInfo.h
  llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
  llvm/lib/CodeGen/AsmPrinter/DwarfDebug.h
  llvm/lib/DebugInfo/DWARF/DWARFExpression.cpp
  llvm/lib/MC/MCDwarf.cpp
  llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCAsmInfo.cpp
  llvm/test/DebugInfo/AMDGPU/cfi.ll
  llvm/test/DebugInfo/X86/heterogeneous_proposal.s
  llvm/test/MC/ELF/AMDGPU/cfi.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76878.387533.patch
Type: text/x-patch
Size: 18605 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211116/9f5eb13c/attachment-0001.bin>


More information about the llvm-commits mailing list