[PATCH] D53021: lld-link: Use /pdbsourcepath: for more places when present.

[Nico Weber via Phabricator via llvm-commits]

thakis added inline comments.


================
Comment at: COFF/PDB.cpp:226
+    // FIXME: make_absolute() should accept sys::path::Style::windows.
+    // Then we should pass that here.
+    sys::fs::make_absolute(FileName);
----------------
Hm, thinking about this a bit more it's not clear what passing a style to make_absolute() should do I suppose :-) .pdb files should contain Windows paths. PDBs created on a non-Windows host without passing /pdbsourcepath: don't, but it's not clear what else we could do -- cwd on a non-Windows host is a non-Windows base path. Maybe I should update the comment that this is just weird in cross builds and that people should pass /pdbsourcepath: there.


https://reviews.llvm.org/D53021