fix Issue 16348 - ICE with package visibility

- fix scopesym parent chain when using semanticScope
- fixes getAccessModule for scope symbols in do bodies,
  try bodies, and else branches

src/statement.d

@@ -192,7 +192,9 @@ public:
     // Same as semanticNoScope(), but do create a new scope
     final Statement semanticScope(Scope* sc, Statement sbreak, Statement scontinue)
     {
-        Scope* scd = sc.push();
+        auto sym = new ScopeDsymbol();
+        sym.parent = sc.scopesym;
+        Scope* scd = sc.push(sym);
         if (sbreak)
             scd.sbreak = sbreak;
         if (scontinue)

test/compilable/imports/test16348.d

@@ -0,0 +1,6 @@
+module mypackage.bar;
+
+package bool bar()
+{
+    return false;
+}

test/compilable/test16348.d

@@ -0,0 +1,13 @@
+// EXTRA_SOURCES: imports/test16348.d
+module mypackage.foo;
+
+void bug()
+{
+    // removing the if-else also removes the segfault
+    if (true) {}
+    else
+    {
+        import mypackage.bar;
+        auto b = bar();
+    }
+}