New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Listeners perf #32495
Listeners perf #32495
Conversation
|
||
// profile create views (run templates in creation mode) | ||
console.profile('create listeners'); | ||
for (let i = 0; i < 500000; i++) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey can we start using A better way to write perf tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, it is using the micro-harness right now.
1c9e0a1
to
092f866
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, one nit
092f866
to
3f21f13
Compare
Before this refactoring we had 2 utility functions to check if a given TNode has matching directives. This PR leaves just one such function (one that does less memory read).
Before this refactoring we had 2 utility functions to check if a given TNode has matching directives. This PR leaves just one such function (one that does less memory read). PR Close #32495
Before this refactoring we had 2 utility functions to check if a given TNode has matching directives. This PR leaves just one such function (one that does less memory read). PR Close angular#32495
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This PR introduces a new benchmark focused on listeners creation. It also contains small refactoring and performance improvement. Go over this PR commit-by-commit for easier review