Only close an upvalue for a local if there actually is one for it.

test/language/closure/unused_closure.wren

@@ -0,0 +1,11 @@
+// This is a regression test. There was a bug where the VM would try to close
+// an upvalue even if the upvalue was never created because the codepath for
+// the closure was not executed.
+
+{
+  var a = "a"
+  if (false) Fn.new { a }
+}
+
+// If we get here, we didn't segfault when a went out of scope.
+System.print("ok") // expect: ok

test/language/closure/unused_later_closure.wren

@@ -0,0 +1,19 @@
+// This is a regression test. When closing upvalues for discarded locals, it
+// wouldn't make sure it discarded the upvalue for the correct stack slot.
+//
+// Here we create two locals that can be closed over, but only the first one
+// actually is. When "b" goes out of scope, we need to make sure we don't
+// prematurely close "a".
+var closure
+
+{
+  var a = "a"
+
+  {
+    var b = "b"
+    closure = Fn.new { a }
+    if (false) Fn.new { b }
+  }
+
+  System.print(closure.call()) // expect: a
+}