Skip to content

layout: Improve paralelization and memory consumption in InitialFlexBoxLayout::new - #46595

Merged
mrobinson merged 3 commits into
servo:mainfrom
Narfinger:improve-paralel
Jul 21, 2026
Merged

mrobinson merged 3 commits into
servo:mainfrom
Narfinger:improve-paralel

Conversation

@Narfinger

@Narfinger Narfinger commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

This cause a 0.6% performance improvement in a test case.

Testing: This change should not change behavior, so existing tests should suffice.

Signed-off-by: Narfinger <[email protected]>
Comment thread components/layout/flexbox/layout.rs
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
@Narfinger
Narfinger marked this pull request as ready for review July 20, 2026 13:08
@Narfinger
Narfinger requested a review from Loirooriol as a code owner July 20, 2026 13:08
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 20, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jul 21, 2026
@mrobinson mrobinson changed the title layout: Improve paralelization and memory consumption in InitialFlexBoxLayout::new layout: Improve paralelization and memory consumption in InitialFlexBoxLayout::new Jul 21, 2026
@mrobinson

Copy link
Copy Markdown
Member

Updating the PR description (which was incomplete) and now sending this to the MQ.

@mrobinson
mrobinson added this pull request to the merge queue Jul 21, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 21, 2026
Merged via the queue into servo:main with commit b1ae367 Jul 21, 2026
36 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 21, 2026
@Narfinger
Narfinger deleted the improve-paralel branch August 7, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants