[PATCH] D95772: [DebugInfo] Normalize paths by removing unnecessary dots

2021-02-11 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D95772#2534103 , @dblaikie wrote: > Does this address @rnk's feedback about symlinks? ( > https://reviews.llvm.org/D87657#2296028 ) It doesn't, I think if we want to handle that case, we'd need to use `sys::fs::real_path` rath

[PATCH] D95772: [DebugInfo] Normalize paths by removing unnecessary dots

2021-02-01 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a subscriber: rnk. dblaikie added a comment. Does this address @rnk's feedback about symlinks? ( https://reviews.llvm.org/D87657#2296028 ) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95772/new/ https://reviews.llvm.org/D95772 ___

[PATCH] D95772: [DebugInfo] Normalize paths by removing unnecessary dots

2021-01-31 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added reviewers: dblaikie, jhenderson. phosek requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This was suggested in https://reviews.llvm.org/D87657 as a better alternative which doesn't require having to