Skip to content

Commit 42980f1

Browse files
committed
---
yaml --- r: 347246 b: refs/heads/master c: 501fa43 h: refs/heads/master
1 parent c459bc8 commit 42980f1

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
refs/heads/master: feaf563ba666df7b40592f9bf6e44902091f9be6
2+
refs/heads/master: 501fa43c4d2713f6d20a4e9047b781d40eaab91a
33
refs/heads/master-next: 203b3026584ecad859eb328b2e12490099409cd5
44
refs/tags/osx-passed: b6b74147ef8a386f532cf9357a1bde006e552c54
55
refs/tags/swift-2.2-SNAPSHOT-2015-12-01-a: 6bb18e013c2284f2b45f5f84f2df2887dc0f7dea

trunk/lib/Basic/OutputFileMap.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,7 @@ void OutputFileMap::write(llvm::raw_ostream &os,
123123
}
124124

125125
os << "\n";
126+
// DenseMap is unordered. If you write a test, please sort the output.
126127
for (auto &typeAndOutputPath : *outputMap) {
127128
file_types::ID type = typeAndOutputPath.getFirst();
128129
StringRef output = typeAndOutputPath.getSecond();

trunk/test/Driver/Inputs/filelists/check-filelist-abc.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,8 @@
5656
supplementaryOutputMapFile = \
5757
sys.argv[sys.argv.index('-supplementary-output-file-map') + 1]
5858
with open(supplementaryOutputMapFile, 'r') as f:
59-
lines = f.readlines()
59+
# The output is in DenseMap order, which is unstable, therefore sort.
60+
lines = sorted(f.readlines())
6061
for line in lines:
6162
print("Supplementary", line.rstrip())
6263

trunk/test/Driver/filelists.swift

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,17 @@
66

77
// CHECK-NOT: Handled
88
// CHECK: Handled a.swift
9-
// CHECK-NEXT: Supplementary "./a.swift":
10-
// CHECK-NEXT: Supplementary swiftmodule: "./a.swiftmodule"
119
// CHECK-NEXT: Supplementary swiftdoc: "./a.swiftdoc"
10+
// CHECK-NEXT: Supplementary swiftmodule: "./a.swiftmodule"
11+
// CHECK-NEXT: Supplementary "./a.swift":
1212
// CHECK-NEXT: Handled b.swift
13-
// CHECK-NEXT: Supplementary "./b.swift":
14-
// CHECK-NEXT: Supplementary swiftmodule: "./b.swiftmodule"
1513
// CHECK-NEXT: Supplementary swiftdoc: "./b.swiftdoc"
14+
// CHECK-NEXT: Supplementary swiftmodule: "./b.swiftmodule"
15+
// CHECK-NEXT: Supplementary "./b.swift":
1616
// CHECK-NEXT: Handled c.swift
17-
// CHECK-NEXT: Supplementary "./c.swift":
18-
// CHECK-NEXT: Supplementary swiftmodule: "./c.swiftmodule"
1917
// CHECK-NEXT: Supplementary swiftdoc: "./c.swiftdoc"
18+
// CHECK-NEXT: Supplementary swiftmodule: "./c.swiftmodule"
19+
// CHECK-NEXT: Supplementary "./c.swift":
2020
// CHECK-NEXT: Handled modules
2121
// CHECK-NOT: Handled
2222

@@ -26,8 +26,8 @@
2626

2727
// CHECK-WMO-NOT: Handled
2828
// CHECK-WMO: Handled all
29-
// CHECK-WMO: Supplementary "{{.*}}/a.swift":
3029
// CHECK-WMO: Supplementary object: "main.o"
30+
// CHECK-WMO: Supplementary "{{.*}}/a.swift":
3131
// CHECK-WMO-NOT: output
3232
// CHECK-WMO-NOT: Handled
3333

@@ -43,12 +43,12 @@
4343

4444
// CHECK-WMO-THREADED-NOT: Handled
4545
// CHECK-WMO-THREADED: Handled all
46-
// CHECK-WMO-THREADED-NEXT: Supplementary "{{.*}}/a.swift":
4746
// CHECK-WMO-THREADED-NEXT: Supplementary {{object|llvm-bc}}: "{{.*}}/a.{{o|bc}}"
48-
// CHECK-WMO-THREADED-NEXT: Supplementary "{{.*}}/b.swift":
4947
// CHECK-WMO-THREADED-NEXT: Supplementary {{object|llvm-bc}}: "{{.*}}/b.{{o|bc}}"
50-
// CHECK-WMO-THREADED-NEXT: Supplementary "{{.*}}/c.swift":
5148
// CHECK-WMO-THREADED-NEXT: Supplementary {{object|llvm-bc}}: "{{.*}}/c.{{o|bc}}"
49+
// CHECK-WMO-THREADED-NEXT: Supplementary "{{.*}}/a.swift":
50+
// CHECK-WMO-THREADED-NEXT: Supplementary "{{.*}}/b.swift":
51+
// CHECK-WMO-THREADED-NEXT: Supplementary "{{.*}}/c.swift":
5252
// CHECK-WMO-THREADED-NEXT: ...with output!
5353
// CHECK-WMO-THREADED-NOT: Handled
5454

0 commit comments

Comments
 (0)