Merge pull request #551 from davidungar/small-opt

David Ungar · web-flow · commit a8cbf92d1c67 · 2021-03-18T21:48:07.000-07:00
[Incremental, Optimization] Only verifyUseIsOK in a debug build.
diff --git a/Sources/SwiftDriver/IncrementalCompilation/ModuleDependencyGraphParts/NodeFinder.swift b/Sources/SwiftDriver/IncrementalCompilation/ModuleDependencyGraphParts/NodeFinder.swift
@@ -131,7 +131,7 @@ extension ModuleDependencyGraph.NodeFinder {
   
   /// record def-use, return if is new use
   mutating func record(def: DependencyKey, use: Graph.Node) -> Bool {
-    verifyUseIsOK(use)
+    assert(verifyUseIsOK(use))
     return usesByDef.insertValue(use, forKey: def)
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ extension ModuleDependencyGraph.NodeFinder {`
`131`	`131`
`132`	`132`	`/// record def-use, return if is new use`
`133`	`133`	`mutating func record(def: DependencyKey, use: Graph.Node) -> Bool {`
`134`		`- verifyUseIsOK(use)`
	`134`	`+ assert(verifyUseIsOK(use))`
`135`	`135`	`return usesByDef.insertValue(use, forKey: def)`
`136`	`136`	`}`
`137`	`137`	`}`