[PATCH] D65179: Changing representation of .cv_def_range directives in Codeview debug info assembly format for better readability

NILANJANA BASU via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 17:19:36 PDT 2019


nilanjana_basu updated this revision to Diff 212935.
nilanjana_basu marked 4 inline comments as done.
nilanjana_basu added a comment.

1. Removed the functionality for the unimplemented (in CodeView) flag CVDR_DEFRANGE in the MCAsmStreamer.
2. Added new negative test cases for all errors reported for the current change
3. Fixed test cases involving CodeView def_range in MC/COFF


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65179

Files:
  llvm/include/llvm/DebugInfo/CodeView/SymbolRecord.h
  llvm/include/llvm/MC/MCStreamer.h
  llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
  llvm/lib/DebugInfo/CodeView/SymbolDumper.cpp
  llvm/lib/DebugInfo/CodeView/SymbolRecordMapping.cpp
  llvm/lib/MC/MCAsmStreamer.cpp
  llvm/lib/MC/MCParser/AsmParser.cpp
  llvm/lib/MC/MCStreamer.cpp
  llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp
  llvm/test/DebugInfo/COFF/fp-stack.ll
  llvm/test/DebugInfo/COFF/local-variable-gap.ll
  llvm/test/DebugInfo/COFF/local-variables.ll
  llvm/test/DebugInfo/COFF/nrvo.ll
  llvm/test/DebugInfo/COFF/pieces.ll
  llvm/test/DebugInfo/COFF/register-variables.ll
  llvm/test/MC/COFF/cv-def-range-align.s
  llvm/test/MC/COFF/cv-def-range-errors.s
  llvm/test/MC/COFF/cv-def-range-gap.s
  llvm/test/MC/COFF/cv-def-range.s
  llvm/tools/llvm-pdbutil/MinimalSymbolDumper.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65179.212935.patch
Type: text/x-patch
Size: 33606 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190802/e0339782/attachment.bin>


More information about the llvm-commits mailing list