Skip to content

Improve coverage of the compiler source code#10902

Open
bjorng wants to merge 3 commits intoerlang:masterfrom
bjorng:bjorn/compiler/coverage
Open

Improve coverage of the compiler source code#10902
bjorng wants to merge 3 commits intoerlang:masterfrom
bjorng:bjorn/compiler/coverage

Conversation

@bjorng
Copy link
Contributor

@bjorng bjorng commented Mar 24, 2026

No description provided.

bjorng added 3 commits March 23, 2026 07:22
@bjorng
Remove vestiges of set_tuple_element instruction
c959a2a 
In a144913, the compiler would stop emitting the
`set_tuple_element` instruction. However, some vestiges
remained.
@bjorng
beam_call_types: Simplify type analysis for is_record/3
5429b6c 
This change will also improve code coverage.
@bjorng
beam_ss_pre_codegen: Remove unreachable optimization
7cfbf12 
Remove an optimization that can no longer be reached since the
introduction of the `bs_match` instruction in 4f0ec73.
@bjorng bjorng requested review from jhogberg and lucioleKi March 24, 2026 05:10
@bjorng bjorng self-assigned this Mar 24, 2026
@bjorng bjorng added team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI labels Mar 24, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 24, 2026 •
edited
Loading

CT Test Results

    2 files    335 suites   8m 52s ⏱️
  871 tests   865 ✅ 6 💤 0 ❌
5 719 runs  5 713 ✅ 6 💤 0 ❌

Results for commit 7cfbf12.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucioleKi lucioleKi lucioleKi approved these changes

@jhogberg jhogberg Awaiting requested review from jhogberg

Assignees

@bjorng bjorng

Labels

team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bjorng @lucioleKi