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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to