[PATCH] D20138: Refactor CodeView TypeRecord visitors to use common structures

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Tue May 10 17:27:14 PDT 2016


zturner updated this revision to Diff 56842.
zturner added a comment.

I'm thinking I can move the deserialize methods to a cpp file in a followup, because I want to try reducing the amount of boilerplate with some templates, and it's not clear how that will turn out.  If i'm lucky I can get a lot of the deserialize methods to disappear.


http://reviews.llvm.org/D20138

Files:
  include/llvm/DebugInfo/CodeView/CVTypeVisitor.h
  include/llvm/DebugInfo/CodeView/CodeView.h
  include/llvm/DebugInfo/CodeView/TypeRecord.h
  include/llvm/DebugInfo/CodeView/TypeRecords.def
  include/llvm/DebugInfo/CodeView/TypeStream.h
  include/llvm/DebugInfo/CodeView/TypeTableBuilder.h
  lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
  lib/DebugInfo/CodeView/FieldListRecordBuilder.cpp
  lib/DebugInfo/CodeView/TypeDumper.cpp
  lib/DebugInfo/CodeView/TypeStream.cpp
  lib/DebugInfo/CodeView/TypeTableBuilder.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20138.56842.patch
Type: text/x-patch
Size: 105322 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160511/1bbd43b0/attachment-0001.bin>


More information about the llvm-commits mailing list