Does cast to sumtype need deep copy in cgen?

[kbkpbot]

It seems current implement in cgen, to_sumtype only do a memdup, does this need to be a deep copy?

struct v__ast__Ident {
        v__ast__OrExpr or_expr;
        Array_v__ast__Type concrete_types;
        v__ast__ScopeObject obj;
        string mod;
        string name;
        string full_name;
        string cached_name;
        v__ast__IdentInfo info;
        v__ast__Scope* scope;
        v__token__Pos pos;
        v__token__Pos mut_pos;
        v__token__Kind tok_kind : 7;
        v__ast__Language language : 4;
        v__ast__IdentKind kind : 3;
        bool comptime : 1;
        bool is_mut : 1;
        bool ct_expr : 1;
};

v__ast__Expr v__ast__Ident_to_sumtype_v__ast__Expr(v__ast__Ident* x) {
        v__ast__Ident* ptr = memdup(x, sizeof(v__ast__Ident));
        return (v__ast__Expr){ ._v__ast__Ident = ptr, ._typ = _v_type_idx_v__ast__Ident()};
}

for example, there are some fields in v__ast__Ident are pointers, some embedded struct fields are pointers too.

or just make a pointer to x?

v__ast__Expr v__ast__Ident_to_sumtype_v__ast__Expr(v__ast__Ident* x) {
        return (v__ast__Expr){ ._v__ast__Ident = x, ._typ = _v_type_idx_v__ast__Ident()};
}