Skip to content
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

Default to using specialized implementation for some builtins. #1171

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Jan 10, 2025

Default to using specialized implementation for some builtins.

Applies to !/@not_strictly_false/==/!=/@in.

This avoids some of the overhead for function dispatch at the cost of duplicating a subset of that logic and restricting the ability to customize them.

Note: the fast builtins assume heterogeneous equality behavior. If not enabled, the runtime will fall back to the provided equals functions, instead of the new builtins step.

@copybara-service copybara-service bot force-pushed the test_713762977 branch 3 times, most recently from 4b74ed4 to 26f9bb8 Compare January 21, 2025 18:07
Applies to !/@not_strictly_false/==/!=/@in.

This avoids some of the overhead for function dispatch at the cost of duplicating a subset of that logic and restricting the ability to customize them.

Note: the fast builtins assume heterogeneous equality behavior. If not enabled, the runtime will fall back to the provided equals functions, instead of the new builtins step.
PiperOrigin-RevId: 717979767
@copybara-service copybara-service bot merged commit 3aef50a into master Jan 21, 2025
2 checks passed
@copybara-service copybara-service bot deleted the test_713762977 branch January 21, 2025 18:39
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.

1 participant