Merge pull request #4121 from dotty-staging/fix/tasty-bind-owner

Fix ownership of symbols unpickled inside Bind nodes

Author: smarter

compiler/src/dotty/tools/dotc/core/tasty/TreeUnpickler.scala

@@ -144,10 +144,11 @@ class TreeUnpickler(reader: TastyReader,
             else {
               for (i <- 0 until nrefs) readNat()
               if (tag == BIND) {
-                buf += new OwnerTree(start, tag, fork, end)
-                goto(end)
+                // a Bind is never the owner of anything, so we set `end = start`
+                buf += new OwnerTree(start, tag, fork, end = start)
               }
-              else scanTrees(buf, end)
+
+              scanTrees(buf, end)
             }
           }
           else if (tag >= firstNatASTTreeTag) { readNat(); scanTree(buf) }

compiler/test/dotty/tools/dotc/FromTastyTests.scala

@@ -91,10 +91,6 @@ class FromTastyTests extends ParallelTesting {
          // Closure type miss match
          "eff-dependent.scala",
 
-         // Unpickling tree without owner
-         "patmat-bind-typed.scala",
-         "t8395.scala",
-
          // Issue unpickling universes
          "phantom-decls-1.scala",
          "phantom-decls-3.scala",