[PATCH] D63662: Changing CodeView Debug info Type record output

[NILANJANA BASU via Phabricator via llvm-commits]

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

Removed redundant tests (that did not involve type records). Added a few meaningful comments, & removed duplicate code as per reviewer suggestions.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63662

Files:
  llvm/include/llvm/DebugInfo/CodeView/CVTypeVisitor.h
  llvm/include/llvm/DebugInfo/CodeView/CodeViewRecordIO.h
  llvm/include/llvm/DebugInfo/CodeView/TypeRecordMapping.h
  llvm/include/llvm/DebugInfo/CodeView/TypeVisitorCallbackPipeline.h
  llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
  llvm/lib/DebugInfo/CodeView/CVTypeVisitor.cpp
  llvm/lib/DebugInfo/CodeView/CodeViewRecordIO.cpp
  llvm/lib/DebugInfo/CodeView/TypeRecordMapping.cpp
  llvm/test/DebugInfo/COFF/anonymous-struct.ll
  llvm/test/DebugInfo/COFF/big-type.ll
  llvm/test/DebugInfo/COFF/class-options-common.ll
  llvm/test/DebugInfo/COFF/const-unnamed-member.ll
  llvm/test/DebugInfo/COFF/defer-complete-type.ll
  llvm/test/DebugInfo/COFF/enum-co.ll
  llvm/test/DebugInfo/COFF/enum.ll
  llvm/test/DebugInfo/COFF/function-options.ll
  llvm/test/DebugInfo/COFF/globals.ll
  llvm/test/DebugInfo/COFF/inheritance.ll
  llvm/test/DebugInfo/COFF/inlining-padding.ll
  llvm/test/DebugInfo/COFF/inlining-same-name.ll
  llvm/test/DebugInfo/COFF/inlining.ll
  llvm/test/DebugInfo/COFF/lambda.ll
  llvm/test/DebugInfo/COFF/long-type-name.ll
  llvm/test/DebugInfo/COFF/nested-types.ll
  llvm/test/DebugInfo/COFF/retained-types.ll
  llvm/test/DebugInfo/COFF/scopes.ll
  llvm/test/DebugInfo/COFF/static-methods.ll
  llvm/test/DebugInfo/COFF/type-quals.ll
  llvm/test/DebugInfo/COFF/types-array-advanced.ll
  llvm/test/DebugInfo/COFF/types-array-unsized.ll
  llvm/test/DebugInfo/COFF/types-array.ll
  llvm/test/DebugInfo/COFF/types-basic.ll
  llvm/test/DebugInfo/COFF/types-calling-conv.ll
  llvm/test/DebugInfo/COFF/types-cvarargs.ll
  llvm/test/DebugInfo/COFF/types-data-members.ll
  llvm/test/DebugInfo/COFF/types-empty-member-fn.ll
  llvm/test/DebugInfo/COFF/types-method-ref-qualifiers.ll
  llvm/test/DebugInfo/COFF/types-nested-class.ll
  llvm/test/DebugInfo/COFF/types-non-virtual-methods.ll
  llvm/test/DebugInfo/COFF/types-ptr-to-member.ll
  llvm/test/DebugInfo/COFF/types-recursive-struct.ll
  llvm/test/DebugInfo/COFF/unnamed.ll
  llvm/test/DebugInfo/COFF/vftables.ll
  llvm/test/DebugInfo/COFF/virtual-method-kinds.ll
  llvm/test/DebugInfo/COFF/virtual-methods.ll
  llvm/test/DebugInfo/COFF/vtable-optzn-array.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63662.207460.patch
Type: text/x-patch
Size: 50737 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190702/071797c5/attachment.bin>