Is Vec<Box<T>> redundant?

[Austaras]

Would Vec be enough?

[kdy1]

I think it's not redundant because Box<Expr> reduces the size of types containing it.

[Austaras]

But is there really a reason to reduce the size of Vec?

[kdy1]

I can't remember the exact reason, but I think it was related to memmove.
Also, this change requires some profiling.

[Austaras]

Oh, maybe it will cost less when move a large struct/enum to a vec if wrapped in a Box.

[kdy1]

I remember the reasoning.
Vec<Box<Expr>> makes reallocation of vector faster, which is quite common for Expr in swc.