[PATCH] D45213: [COFF][LLD] Add link support for precompiled headers .objs

[Neeraj K. Singh via Phabricator via llvm-commits]

neerajksingh accepted this revision.
neerajksingh added a comment.
This revision is now accepted and ready to land.

I just want to mention as a heads-up: there will later be some complexities in this PCH finding code when the /deterministic flags are added to public builds of cl.exe. PCH files paths might be replaced with environment variables that will need to be expanded when acquiring a PDB.

FYI, in the MS linker, we find out early on that a file contains precompiled type information from the presence of the debug$P section and mostly process those files before processing the non-PCT files.  That avoids some need to search when acquiring the PCH data.


https://reviews.llvm.org/D45213