aaron.ballman added a comment.

In https://reviews.llvm.org/D33788#771504, @akyrtzi wrote:

> Getting the real path is notoriously slow (at least it's horrible in OSX, not 
> sure about linux). Since this is about dropping the '/../' part, could we do 
> some simple canonicalization of removing the dots ? Not sure if there is an 
> existing function that does that.


Given the fact that this path is cached, so you only need to do the slow path 
one time, are you sure that's a practical concern? I'm unaware of any other API 
that canonicalizes the path, which is what users of this API are going to 
expect.


https://reviews.llvm.org/D33788



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

Reply via email to