[clang-tools-extra] r332612 - Second attempt to fix clang-move tests broken by r332590 on windows
Eric Liu via cfe-commits
cfe-commits at lists.llvm.org
Thu May 17 07:59:15 PDT 2018
Author: ioeric
Date: Thu May 17 07:59:15 2018
New Revision: 332612
URL: http://llvm.org/viewvc/llvm-project?rev=332612&view=rev
Log:
Second attempt to fix clang-move tests broken by r332590 on windows
http://lab.llvm.org:8011/builders/clang-x86-windows-msvc2015/builds/12010
Add back a path conversion removed in a previous attempt. I removed it
because it didn't seem necessary. Added it back to see if this would fix
the bot.
Modified:
clang-tools-extra/trunk/clang-move/ClangMove.cpp
Modified: clang-tools-extra/trunk/clang-move/ClangMove.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-move/ClangMove.cpp?rev=332612&r1=332611&r2=332612&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-move/ClangMove.cpp (original)
+++ clang-tools-extra/trunk/clang-move/ClangMove.cpp Thu May 17 07:59:15 2018
@@ -64,6 +64,8 @@ AST_MATCHER_P(CXXMethodDecl, ofOutermost
std::string CleanPath(StringRef PathRef) {
llvm::SmallString<128> Path(PathRef);
llvm::sys::path::remove_dots(Path, /*remove_dot_dot=*/true);
+ // FIXME: figure out why this is necessary.
+ llvm::sys::path::native(Path);
return Path.str();
}
More information about the cfe-commits
mailing list