[PATCH] D34432: [PDB] Add symbols to the PDB

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 21 09:20:39 PDT 2017


rnk marked 4 inline comments as done.
rnk added inline comments.


================
Comment at: lld/COFF/PDB.cpp:142-144
+        log("ignoring symbol record in " + File->getName() +
+            " with bad type index 0x" + utohexstr(TI.getIndex()));
+        return false;
----------------
ruiu wrote:
> You probably want to use `fatal` as this code handles broken input files.
It's more likely that our understanding of the record is broken that it is for the record to actually be corrupt. This line actually fires today for files compiled with /Zi, because there's something wrong with our type server handling. libcmt.lib was compiled with /Zi, so any program using the CRT hits this.

I'd rather recover by skipping the record for now. It doesn't result in a broken program, just a less useful PDB.


https://reviews.llvm.org/D34432





More information about the llvm-commits mailing list