Skip to content

script: Use NoGC for note_dirty_descendants - #46659

Merged
Gae24 merged 2 commits into
servo:mainfrom
Narfinger:move-owner_doc-to-no_gc
Jul 20, 2026
Merged

Gae24 merged 2 commits into
servo:mainfrom
Narfinger:move-owner_doc-to-no_gc

Conversation

@Narfinger

Copy link
Copy Markdown
Contributor

Use no_gc for note_dirty_descendants. This should take 0.4% runtime cost away from script in a small slice.

Testing: Describe the new automated tests that cover this change or explain why it doesn't require tests.
Fixes: Link to an issue this pull request fixes or remove this line if there is no issue.

@Narfinger
Narfinger requested a review from gterzian as a code owner July 20, 2026 14:17
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 20, 2026
Comment thread components/script/dom/node/node.rs Outdated
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jul 20, 2026
Co-authored-by: Gae24 <[email protected]>
Signed-off-by: Narfinger <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 20, 2026
@Gae24
Gae24 enabled auto-merge July 20, 2026 16:59
@Gae24
Gae24 added this pull request to the merge queue Jul 20, 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 20, 2026
Merged via the queue into servo:main with commit 3d85484 Jul 20, 2026
34 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 20, 2026
@Narfinger
Narfinger deleted the move-owner_doc-to-no_gc branch July 31, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants