[PATCH] D22693: More strongly separate the PDB reading interfaces and PDB writing interfaces

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 15:05:07 PDT 2016


ruiu added a comment.

At a high level, it looks good to me, but as I probably mentioned earlier, I have a little concern that we are having an API for PDB that is too complete at this moment. The COFF linker is 7000 lines long in total, and if we keep this pace, we'll probably end up having a PDB library which is larger than the linker. Even if we'd eventually need it, I'm more comfortable if we do as-needed basis.


https://reviews.llvm.org/D22693





More information about the llvm-commits mailing list