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

erts: Make sure to tag orig with literal ptr #9349

Merged

Conversation

garazdawi
Copy link
Contributor

No description provided.

@garazdawi garazdawi added the fix label Jan 27, 2025
@garazdawi garazdawi requested a review from jhogberg January 27, 2025 22:40
@garazdawi garazdawi self-assigned this Jan 27, 2025
@garazdawi garazdawi linked an issue Jan 27, 2025 that may be closed by this pull request
Copy link
Contributor

github-actions bot commented Jan 27, 2025

CT Test Results

    3 files    143 suites   48m 47s ⏱️
1 597 tests 1 548 ✅ 49 💤 0 ❌
2 341 runs  2 267 ✅ 74 💤 0 ❌

Results for commit c3c2d0b.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@garazdawi garazdawi force-pushed the lukas/erts/fix-persistent-term-literal-tag branch from 7c3f7b8 to 05c5bb0 Compare January 28, 2025 08:53
jhogberg
jhogberg previously approved these changes Jan 28, 2025
@garazdawi garazdawi added the testing currently being tested, tag is used by OTP internal CI label Jan 28, 2025
We assert that if we are in a literal range, the literal tag
should also be set. This will only be true in the debug build,
but as asserts are only active in debug builds that is fine.
@garazdawi garazdawi force-pushed the lukas/erts/fix-persistent-term-literal-tag branch from 05c5bb0 to 8324f3e Compare January 28, 2025 14:30
@garazdawi garazdawi merged commit e060f28 into erlang:maint Jan 29, 2025
25 checks passed
@garazdawi garazdawi added this to the OTP-27.3 milestone Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Windows] Beam crash related to DETS or persistent_term
2 participants