Commit 8396c5b8 authored by Hans Wennborg's avatar Hans Wennborg
Browse files

Merging r292032:

------------------------------------------------------------------------
r292032 | yrnkrn | 2017-01-14 13:12:08 -0800 (Sat, 14 Jan 2017) | 4 lines

Fix PR31644 introduced by r287138 and add a regression test.
Thanks Dimitry Andric for the report and fix!


------------------------------------------------------------------------


git-svn-id: https://llvm.org/svn/llvm-project/cfe/branches/release_40@292311 91177308-0d34-0410-b5e6-96231b3b80d8
parent ed738531
......@@ -447,9 +447,9 @@ void DFGImpl::OutputDependencyFile() {
// Create phony targets if requested.
if (PhonyTarget && !Files.empty()) {
// Skip the first entry, this is always the input file itself.
for (StringRef File : Files) {
for (auto I = Files.begin() + 1, E = Files.end(); I != E; ++I) {
OS << '\n';
PrintFilename(OS, File, OutputFormat);
PrintFilename(OS, *I, OutputFormat);
OS << ":\n";
}
}
......
......@@ -32,5 +32,12 @@
// RUN: FileCheck -check-prefix=TEST5 %s < %t.d
// TEST5: foo $$(bar) b az qu\ ux \ space:
// Test self dependency, PR31644
// RUN: %clang -E -MD -MP -MF %t.d %s
// RUN: FileCheck -check-prefix=TEST6 %s < %t.d
// TEST6: dependencies-and-pp.c
// TEST6-NOT: dependencies-and-pp.c:
// TODO: Test default target without quoting
// TODO: Test default target with quoting
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment