[PATCH] D35092: Use native path syntax when writing PDB module name.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 6 14:54:22 PDT 2017


zturner created this revision.

Previously `lld-link` would write module names containing forward slashes, even on Windows.  This differs from what `link` does, which makes sure all paths are Windows-ized.  This may or may not be necessary for Windows tools to work with clang-cl generated PDBs, but since we don't know, we need to do the same thing.  This issue was also causing diff to think these were two different modules and not comparing their fields, so we now get better output from diff.


https://reviews.llvm.org/D35092

Files:
  lld/COFF/PDB.cpp
  lld/test/COFF/pdb-diff.test
  llvm/tools/llvm-pdbutil/Diff.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35092.105545.patch
Type: text/x-patch
Size: 8201 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170706/d7363d9e/attachment.bin>


More information about the llvm-commits mailing list