Skip to content

Conversation

@MathiasVP
Copy link
Contributor

@MathiasVP MathiasVP commented Sep 15, 2025

This fixes a performance problem when switching C/C++ to the shared guards library.

The Rust CI failure is (obviously) unrelated.

@MathiasVP MathiasVP requested review from aschackmull and Copilot and removed request for Copilot September 15, 2025 16:13
@MathiasVP MathiasVP added the no-change-note-required This PR does not need a change note label Sep 15, 2025
Copy link
Contributor

@aschackmull aschackmull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@geoffw0
Copy link
Contributor

geoffw0 commented Sep 16, 2025

The Rust CI failure is (obviously) unrelated.

Confirmed, sorry about that.

@MathiasVP MathiasVP merged commit fa36d9f into github:main Sep 16, 2025
36 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants