[Issue 22164] New: Rewrite for scope guards produces inconsistent AST

[d-bugmail at puremagic.com]

https://issues.dlang.org/show_bug.cgi?id=22164

          Issue ID: 22164
           Summary: Rewrite for scope guards produces inconsistent AST
           Product: D
           Version: D2
          Hardware: All
                OS: All
            Status: NEW
          Severity: minor
          Priority: P1
         Component: dmd
          Assignee: nobody at puremagic.com
          Reporter: moonlightsentinel at disroot.org

The semantic analysis rewrites scope guards using try-catch statements. But
this process leaves null's in the list of statements instead of fully replacing
the exisitng ScopeGuardStatement (and hence leaves an inconsistent AST).

See the FIXME comments in test/unit/semantic/control_flow.d introduced in
https://github.com/dlang/dmd/pull/12939

--