[move-only] Remove a TODO that says we are erroring on the same variable twice. We are no longer doing that in the specific test.

gottesmm · gottesmm · commit 444701c5a135 · 2023-05-04T12:44:30.000-07:00
diff --git a/test/SILOptimizer/moveonly_addresschecker_diagnostics.swift b/test/SILOptimizer/moveonly_addresschecker_diagnostics.swift
@@ -3079,8 +3079,6 @@ public func closureClassUseAfterConsumeArg(_ argX: inout Klass) {
 }
 
 // We do not support captures of vars by closures today.
-//
-// TODO: Why are we erroring for the same variable twice?
 public func closureCaptureClassUseAfterConsume() {
     var x2 = Klass()
     x2 = Klass()

Original file line number	Diff line number	Diff line change
`@@ -3079,8 +3079,6 @@ public func closureClassUseAfterConsumeArg(_ argX: inout Klass) {`
`3079`	`3079`	`}`
`3080`	`3080`
`3081`	`3081`	`// We do not support captures of vars by closures today.`
`3082`		`-//`
`3083`		`-// TODO: Why are we erroring for the same variable twice?`
`3084`	`3082`	`public func closureCaptureClassUseAfterConsume() {`
`3085`	`3083`	`var x2 = Klass()`
`3086`	`3084`	`x2 = Klass()`