-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Add Impeller complex layout impeller benchmarks. #139490
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
Add Impeller complex layout impeller benchmarks. #139490
Conversation
zanderso
left a comment
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
| /dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller__timeline_summary.dart @jonahwilliams @flutter/engine | ||
| /dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller_gles__timeline_summary.dart @jonahwilliams @flutter/engine |
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.
Nit: order these entries alphabetically.
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.
like .. all of the entries? or just these last two?
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.
Oh, I didn't know they are already out of order 😂
Feel free to land here then. I will create a following PR to order the whole list.
…r-xu/flutter into fix/gridview_delegate * 'fix/gridview_delegate' of https://github.com/hello-coder-xu/flutter: (162 commits) add gridview children view fix gridview delegate mainAxisExtent Enable TapRegion to detect all mouse button click (flutter#136799) [flutter_tools] build ios-frameworks: option to exclude plugin frameworks from the build (flutter#129739) Roll Flutter Engine from b87c1b3 to eee8aeb (1 revision) (flutter#139495) Add Impeller complex layout impeller benchmarks. (flutter#139490) Roll Flutter Engine from cf12246 to b87c1b3 (3 revisions) (flutter#139491) Remove InkSparkle.constantTurbulenceSeedSplashFactory usage (flutter#139481) Roll Packages from bc72d15 to 5d9874f (9 revisions) (flutter#139480) Roll Flutter Engine from cc9a72b to cf12246 (1 revision) (flutter#139468) Roll Flutter Engine from 69cbf87 to cc9a72b (1 revision) (flutter#139466) Roll Flutter Engine from 05e7f46 to 69cbf87 (1 revision) (flutter#139453) Roll Flutter Engine from e782ab1 to 05e7f46 (1 revision) (flutter#139439) Roll Flutter Engine from 80a2fd7 to e782ab1 (1 revision) (flutter#139437) Roll Flutter Engine from 7ff24be to 80a2fd7 (1 revision) (flutter#139436) Roll Flutter Engine from cfabe42 to 7ff24be (1 revision) (flutter#139433) Roll Flutter Engine from f0122c3 to cfabe42 (1 revision) (flutter#139423) Roll Flutter Engine from f23c33f to f0122c3 (1 revision) (flutter#139422) Roll Flutter Engine from d441f08 to f23c33f (2 revisions) (flutter#139421) Roll Flutter Engine from 27d37db to d441f08 (1 revision) (flutter#139419) ...
These benchmarks do a nice job of showing CPU overhead, which is useful for many of the microoptimizations we've been doing.