[PATCH] D31267: [codeview] Move type index remapping logic to type merger
Zachary Turner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 22 16:33:25 PDT 2017
zturner accepted this revision.
zturner added inline comments.
================
Comment at: lib/DebugInfo/CodeView/TypeStreamMerger.cpp:163
+Error TypeStreamMerger::visitKnownRecord(CVType &, ModifierRecord &R) {
+ return writeRecord(R, remapIndex(R.ModifiedType));
+}
----------------
I forgot why we store `LastError` and proceed through the type stream, instead of just returning the error right away and failing. Do you happen to know?
https://reviews.llvm.org/D31267
More information about the llvm-commits
mailing list