[PATCH] D14394: Moving FileManager::removeDotPaths to llvm::sys::path::remove_dots
Sean Silva via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 5 19:46:51 PST 2015
silvas added a subscriber: silvas.
================
Comment at: test/Frontend/dependency-gen.c:19
@@ -18,3 +18,3 @@
// RUN: %clang -MD -MF - %s -fsyntax-only -I a/b/. | FileCheck -check-prefix=CHECK-FIVE %s
-// CHECK-FIVE: {{ }}a/b/.{{[/\\]}}x.h
+// CHECK-FIVE: {{ }}a/b{{[/\\]}}x.h
// RUN: cd a/b
----------------
aizatsky wrote:
> kcc wrote:
> > This is an intended change, right? Why?
> Because new function also removes /./ in the middle. It used to be a/b/./x.h and it is now a/b/x.h
To keep this change mechanical, let's just not touch `remove_leading_dotslash` in this patch.
http://reviews.llvm.org/D14394
More information about the llvm-commits
mailing list