[PATCH] D35496: [codeview] Don't use the type visitor to merge types

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

I agree that this is a nice simplification.



================
Comment at: llvm/lib/DebugInfo/CodeView/TypeStreamMerger.cpp:282
+  discoverTypeIndices(Type.RecordData, Refs);
+  bool Success = remapIndices(R, Refs);
+  if (isIdRecord(Type.kind())) {
----------------
What is the meaning of this boolean varaible? Looks like you are not using its value.


https://reviews.llvm.org/D35496