[PATCH] D71508: [DebugInfo] Duplicate file names in debug info

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 16 09:20:42 PST 2019


probinson added a comment.

In D71508#1786122 <https://reviews.llvm.org/D71508#1786122>, @kamleshbhalui wrote:

> In D71508#1785767 <https://reviews.llvm.org/D71508#1785767>, @probinson wrote:
>
> > Do we have a similar problem if the filespec has an embedded ./ or ../ in it?
>
>
> problems  occur only when filespec starts with ./ otherwise it's fine.


So do other kinds of paths get canonicalized elsewhere?  I'm thinking if there's already a place that undoes embedded .. then it should handle leading . as well.

Is a leading .. okay or a problem?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71508/new/

https://reviews.llvm.org/D71508





More information about the cfe-commits mailing list