Skip to content

beam_ssa_opt: Fix block tracking in bsm_shortcut #10048

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

Conversation

jhogberg
Copy link
Contributor

Success blocks were not adequately tracked, causing bs_test_tail instructions to be removed when they shouldn't have been.

Fixes GH-10047

@jhogberg jhogberg requested a review from lucioleKi July 15, 2025 10:44
@jhogberg jhogberg self-assigned this Jul 15, 2025
@jhogberg jhogberg added team:VM Assigned to OTP team VM fix testing currently being tested, tag is used by OTP internal CI labels Jul 15, 2025
Copy link
Contributor

github-actions bot commented Jul 15, 2025

CT Test Results

    2 files    324 suites   9m 27s ⏱️
  822 tests   820 ✅ 2 💤 0 ❌
5 445 runs  5 443 ✅ 2 💤 0 ❌

Results for commit b7f18fb.

♻️ 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

potatosalad added a commit to WhatsApp/erlang-argo that referenced this pull request Jul 15, 2025
lucioleKi
lucioleKi previously approved these changes Jul 16, 2025
@jhogberg jhogberg force-pushed the john/compiler/fix-bsm-shortcut/GH-10047/OTP-19711 branch from 2b2e5df to a1d8cc1 Compare July 16, 2025 08:39
Success blocks were not adequately tracked, causing `bs_test_tail`
instructions to be removed when they shouldn't have been.

Fixes erlangGH-10047
@jhogberg jhogberg force-pushed the john/compiler/fix-bsm-shortcut/GH-10047/OTP-19711 branch from a1d8cc1 to b7f18fb Compare July 16, 2025 10:37
@rickard-green rickard-green merged commit ed5cb1b into erlang:maint-27 Jul 17, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants