Improve noncopyable closure diags

[kavon]

After polishing the diagnostics a bit, I got some of the closure ones wrong:

struct NonCopyableStruct: ~Copyable {}

var globFn: () -> () = {}
func forceEscaping(_ esc: @escaping () -> ()) {
  globFn = esc
}

func closureDiagnosticsSimple() {
  var s = NonCopyableStruct()
  let f = {
    _ = consume s  // error: missing reinitialization of inout parameter 's' after consume
    s = NonCopyableStruct()
  }
  forceEscaping(f)
  f()
}

There are a few problems here:

1. This shouldn't be called an inout parameter and instead a closure capture.
2. Seems it's not actually possible to reinit to remedy this situation, but only when the closure is escaping. When it is not escaping, it's valid to reconcile this with reinit. We are saying noncopyable '%0' cannot be consumed when captured by a closure when a non-reconcilable consume happens in a closure. So we should specify "an escaping closure" here.

rdar://109908383

[kavon]

@swift-ci smoke test and merge

[kavon]

@swift-ci please smoke test and merge