diff --git a/clang/lib/Tooling/CompilationDatabase.cpp b/clang/lib/Tooling/CompilationDatabase.cpp --- a/clang/lib/Tooling/CompilationDatabase.cpp +++ b/clang/lib/Tooling/CompilationDatabase.cpp @@ -368,8 +368,14 @@ ErrorMsg = "Error while opening fixed database: " + Result.message(); return nullptr; } - std::vector Args{llvm::line_iterator(**File), - llvm::line_iterator()}; + std::vector Args; + for (llvm::StringRef Line : + llvm::make_range(llvm::line_iterator(**File), llvm::line_iterator())) { + // Stray whitespace is almost certainly unintended. + Line = Line.trim(); + if (!Line.empty()) + Args.push_back(Line.str()); + } return std::make_unique( llvm::sys::path::parent_path(Path), std::move(Args)); } diff --git a/clang/test/Tooling/fixed-database.cpp b/clang/test/Tooling/fixed-database.cpp --- a/clang/test/Tooling/fixed-database.cpp +++ b/clang/test/Tooling/fixed-database.cpp @@ -5,7 +5,7 @@ // RUN: cp "%S/Inputs/fixed-header.h" "%t/Include/" // -I flag is relative to %t (where compile_flags is), not Src/. // RUN: echo '-IInclude/' >> %t/compile_flags.txt -// RUN: echo "-Dklazz=class" >> %t/compile_flags.txt +// RUN: echo " -Dklazz=class " >> %t/compile_flags.txt // RUN: echo '-std=c++11' >> %t/compile_flags.txt // RUN: clang-check "%t/Src/test.cpp" 2>&1 // RUN: echo > %t/compile_flags.txt