search

glimmerjs / glimmer-vm

MIT License
1.13k stars 191 forks source link

Streamline append VM implementation #1646

Closed wycats closed 3 weeks ago

wycats commented 3 weeks ago

Attempting to pull out the commit that doesn't seem to have a standalone performance impact so we can land it separately.

NullVoxPopuli commented 3 weeks ago

I ran it twice

image

image

NullVoxPopuli commented 3 weeks ago

We both tested this locally, and perf is fine