NullVoxPopuli
NullVoxPopuli
I'd like to block on on using data attributes for classes instead of classes.
Closing this because it turned out to not be relevant
Closing, because https://github.com/glimmerjs/glimmer-vm/pull/1606 solved a lot
> duration phase estimated improvement -110ms woah, that's nice -- Suppose I should compare with the other versions
The improvement revealed by tracerbench translates all they way through to ember.   
Closing, because https://github.com/glimmerjs/glimmer-vm/pull/1606 solved a lot
Some preliminary improvement here (development mode):  (at the cost of type safety)
In production mode, it looks like we get back to 5.6 performance https://glimmer-vm-testing-3.ember-performance-testing-prod.pages.dev/?benchmarks=%5B%22Render%20complex%20html%20(%40glimmer%2Fcomponent)%22%2C%22Render%20complex%20html%20(template-only)%22%5D&emberVersions=%5B%225.5%22%2C%225.6%22%2C%225.7%22%2C%225.8%22%2C%225.9%22%2C%225.10%22%2C%22ember-canary%22%2C%22ember-canary-custom%22%5D 
> It seems like these were intended to be stripped/strippable in prod/turned into no-op, any idea why that isn’t happening? (or maybe the stripping was just never implemented?) I think...
> https://github.com/glimmerjs/glimmer-vm/pull/1602 it's perhaps not just terser, or we need more inlining or something. Or maybe it's possible that the way the checks are written -- they just aren't strippable....