Merge pull request #13980 from davidungar/PR-18-2

David Ungar · web-flow · commit adce82dfec12 · 2018-01-16T19:12:42.000-08:00
Remove redundant assertion, which is already checked at lower level.
diff --git a/include/swift/Frontend/FrontendOptions.h b/include/swift/Frontend/FrontendOptions.h
@@ -96,7 +96,6 @@ class FrontendInputs {
   std::vector<std::string> getInputFilenames() const {
     std::vector<std::string> filenames;
     for (auto &input : getAllFiles()) {
-      assert(!input.file().empty());
       filenames.push_back(input.file());
     }
     return filenames;
diff --git a/test/Serialization/vtable-function-deserialization.swift b/test/Serialization/vtable-function-deserialization.swift
@@ -1,5 +1,5 @@
 // RUN: %empty-directory(%t)
-// RUN: %target-swift-frontend -emit-module -o Swift.swiftmodule -O -sil-inline-threshold 0 -module-name Swift -module-link-name swiftCore -parse-as-library -parse-stdlib -emit-module %S/Inputs/vtable-function-deserialization-input.swift -o %t/Swift.swiftmodule
+// RUN: %target-swift-frontend -emit-module -O -sil-inline-threshold 0 -module-name Swift -module-link-name swiftCore -parse-as-library -parse-stdlib -emit-module %S/Inputs/vtable-function-deserialization-input.swift -o %t/Swift.swiftmodule
 // RUN: %target-swift-frontend -I %t %s -emit-sil -o - -O -sil-link-all
 
 import Swift

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,6 @@ class FrontendInputs {`
`96`	`96`	`std::vector<std::string> getInputFilenames() const {`
`97`	`97`	`std::vector<std::string> filenames;`
`98`	`98`	`for (auto &input : getAllFiles()) {`
`99`		`- assert(!input.file().empty());`
`100`	`99`	`filenames.push_back(input.file());`
`101`	`100`	`}`
`102`	`101`	`return filenames;`