[PATCH] D31267: [codeview] Move type index remapping logic to type merger

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 16:26:19 PDT 2017


rnk created this revision.
Herald added a subscriber: mgorny.

This removes the 'remapTypeIndices' method on every TypeRecord class. My
original idea was that this would be the beginning of some kind of
generic entry point that would enumerate all of the TypeIndices inside
of a TypeRecord, so that we could write generic graph algorithms for
them without duplicating the knowledge of which fields are type index
fields everywhere. This never happened, and nothing else uses this
method. I need to change the API to deal with merging into IPI streams,
so let's move it into the file that uses it first.


https://reviews.llvm.org/D31267

Files:
  include/llvm/DebugInfo/CodeView/TypeRecord.h
  lib/DebugInfo/CodeView/CMakeLists.txt
  lib/DebugInfo/CodeView/TypeRecord.cpp
  lib/DebugInfo/CodeView/TypeStreamMerger.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31267.92736.patch
Type: text/x-patch
Size: 31194 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170322/0245d05c/attachment-0001.bin>


More information about the llvm-commits mailing list