diff --git a/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp b/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp index 9d21733b83a2..2a5689898a48 100644 --- a/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp +++ b/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp @@ -2,7 +2,6 @@ #include "Core/Transform.h" #include "clang/AST/ASTConsumer.h" #include "clang/AST/DeclGroup.h" -#include "llvm/Support/FileSystem.h" #include "llvm/Support/Process.h" #include "llvm/Support/Path.h" #include "llvm/Support/PathV1.h" @@ -99,24 +98,22 @@ TEST(Transform, Timings) { // The directory used is not important since the path gets mapped to a virtual // file anyway. What is important is that we have an absolute path with which // to use with mapVirtualFile(). - SmallString<128> CurrentDir; - llvm::error_code EC = llvm::sys::fs::current_path(CurrentDir); - EXPECT_FALSE(EC); + llvm::sys::Path FileA = llvm::sys::Path::GetCurrentDirectory(); + std::string CurrentDir = FileA.str(); + FileA.appendComponent("a.cc"); + std::string FileAName = FileA.str(); + llvm::sys::Path FileB = llvm::sys::Path::GetCurrentDirectory(); + FileB.appendComponent("b.cc"); + std::string FileBName = FileB.str(); - SmallString<128> FileA = CurrentDir; - llvm::sys::path::append(FileA, "a.cc"); - - SmallString<128> FileB = CurrentDir; - llvm::sys::path::append(FileB, "b.cc"); - - tooling::FixedCompilationDatabase Compilations(CurrentDir.str(), std::vector()); + tooling::FixedCompilationDatabase Compilations(CurrentDir, std::vector()); std::vector Sources; - Sources.push_back(FileA.str()); - Sources.push_back(FileB.str()); + Sources.push_back(FileAName); + Sources.push_back(FileBName); tooling::ClangTool Tool(Compilations, Sources); - Tool.mapVirtualFile(FileA, "void a() {}"); - Tool.mapVirtualFile(FileB, "void b() {}"); + Tool.mapVirtualFile(FileAName, "void a() {}"); + Tool.mapVirtualFile(FileBName, "void b() {}"); ConsumerFactory Factory; Tool.run(newFrontendActionFactory(&Factory, &T)); @@ -128,13 +125,13 @@ TEST(Transform, Timings) { // The success of the test shouldn't depend on the order of iteration through // timers. llvm::sys::Path FirstFile(I->first); - if (FileA == FirstFile.str()) { + if (FileA == FirstFile) { ++I; - EXPECT_EQ(FileB, llvm::sys::Path(I->first).str()); + EXPECT_EQ(FileB, llvm::sys::Path(I->first)); EXPECT_GT(I->second.getProcessTime(), 0.0); - } else if (FileB == FirstFile.str()) { + } else if (FileB == FirstFile) { ++I; - EXPECT_EQ(FileA, llvm::sys::Path(I->first).str()); + EXPECT_EQ(FileA, llvm::sys::Path(I->first)); EXPECT_GT(I->second.getProcessTime(), 0.0); } else { FAIL() << "Unexpected file name " << I->first << " in timing data.";