[PATCH] D44290: Handle mixed-OS paths in DWARF reader

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 05:08:39 PDT 2018


labath added a comment.

I think that adding a path-style field to dwarf (it would probably have to be a vendor extension) is an excellent idea. However, that won't help us with parsing the existing debug info corpus. This isn't going to be 100% correct (e.g. `c:\foo.cpp` is a valid **relative** path on unix), but I hope nobody actually uses file names where that matters. If someone like that turns up, then we'd probably have to add a some flag about how the user wants the paths to be treated, as there is no way we can figure out the correct behavior automatically.


https://reviews.llvm.org/D44290





More information about the llvm-commits mailing list