Fix duplicate local issue.

dotnet · Aug 21, 2021 · 50605e9 · 50605e9
1 parent adb4c64
commit 50605e9
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/.../Portable/Lowering/LocalRewriter/LocalRewriter_ObjectOrCollectionInitializerExpression.cs b/.../Portable/Lowering/LocalRewriter/LocalRewriter_ObjectOrCollectionInitializerExpression.cs
@@ -207,13 +207,14 @@ private BoundExpression MakeDynamicCollectionInitializer(BoundExpression rewritt
             return MakeCall(null, syntax, rewrittenReceiver, addMethod, rewrittenArguments, argumentRefKindsOpt, initializer.InvokedAsExtensionMethod, initializer.ResultKind, addMethod.ReturnType, temps.ToImmutableAndFree());
         }
 
-        public BoundExpression VisitObjectInitializerMember(BoundObjectInitializerMember node, ref BoundExpression rewrittenReceiver, ArrayBuilder<BoundExpression> sideEffects, ref ArrayBuilder<LocalSymbol>? temps)
+        private BoundExpression VisitObjectInitializerMember(BoundObjectInitializerMember node, ref BoundExpression rewrittenReceiver, ArrayBuilder<BoundExpression> sideEffects, ref ArrayBuilder<LocalSymbol>? temps)
         {
             if (node.MemberSymbol is null)
             {
                 return (BoundExpression)base.VisitObjectInitializerMember(node)!;
             }
 
+            var originalReceiver = rewrittenReceiver;
             var rewrittenArguments = VisitArguments(node.Arguments, node.MemberSymbol, node.ArgsToParamsOpt, node.ArgumentRefKindsOpt, ref rewrittenReceiver, out ArrayBuilder<LocalSymbol>? constructionTemps);
 
             if (constructionTemps != null)
@@ -229,7 +230,7 @@ public BoundExpression VisitObjectInitializerMember(BoundObjectInitializerMember
                 }
             }
 
-            if (rewrittenReceiver is BoundSequence sequence)
+            if (originalReceiver != rewrittenReceiver && rewrittenReceiver is BoundSequence sequence)
             {
                 Debug.Assert(temps != null);
                 temps.AddRange(sequence.Locals);