refactor(transformer/arrow-function): create a new ident instead of c…

…lone

crates/oxc_transformer/src/common/arrow_function_converter.rs

@@ -1008,17 +1008,17 @@ impl<'a> ArrowFunctionConverter<'a> {
         }
 
         Self::adjust_binding_scope(target_scope_id, &arguments_var, ctx);
-        let reference =
+
+        let ident1 =
+            ctx.create_unbound_ident_reference(SPAN, Atom::from("arguments"), ReferenceFlags::Read);
+        let ident2 =
             ctx.create_unbound_ident_reference(SPAN, Atom::from("arguments"), ReferenceFlags::Read);
-        // TODO: We shouldn't be cloning `IdentifierReference` here.
-        // We'll end up with 2 x `IdentifierReference`s with same `ReferenceId`.
-        // I guess we don't have a test that covers this, or transform semantic checker would have flagged it.
-        let mut init = Expression::Identifier(ctx.ast.alloc(reference.clone()));
+        let mut init = Expression::Identifier(ctx.ast.alloc(ident1));
 
         // Top level may doesn't have `arguments`, so we need to check it.
         // `typeof arguments === "undefined" ? void 0 : arguments;`
         if ctx.scopes().root_scope_id() == target_scope_id {
-            let argument = Expression::Identifier(ctx.ast.alloc(reference));
+            let argument = Expression::Identifier(ctx.ast.alloc(ident2));
             let typeof_arguments = ctx.ast.expression_unary(SPAN, UnaryOperator::Typeof, argument);
             let undefined_literal = ctx.ast.expression_string_literal(SPAN, "undefined", None);
             let test = ctx.ast.expression_binary(