Skip to content

compiler: Remove dead code from private-append optimization #6875

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

Merged
merged 1 commit into from
Feb 18, 2023

Conversation

frej
Copy link
Contributor

@frej frej commented Feb 16, 2023

With the decision to not introduce a private append variant of bs_create_bin which gracefully tolerates non-binary string arguments, we can remove many code paths in beam_ssa_private_append.erl which are never reached as we will never try to track and patch the tail field of a cons.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 16, 2023

CT Test Results

       2 files     296 suites   12m 47s ⏱️
   761 tests    759 ✔️ 2 💤 0
4 792 runs  4 790 ✔️ 2 💤 0

Results for commit 1694e70.

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

@bjorng bjorng self-assigned this Feb 16, 2023
@bjorng bjorng added team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI labels Feb 16, 2023
@frej
Copy link
Contributor Author

frej commented Feb 17, 2023

It turns out that this patch depends on #6855 and maybe #6862, will rebase when they have been merged.

@bjorng bjorng removed the testing currently being tested, tag is used by OTP internal CI label Feb 17, 2023
@bjorng
Copy link
Contributor

bjorng commented Feb 17, 2023

I have merged your other three pull requests. You can rebase now.

With the decision to not introduce a private append variant of
`bs_create_bin` which gracefully tolerates non-binary string
arguments, we can remove many code paths in
`beam_ssa_private_append.erl` which are never reached as we will never
try to track and patch the tail field of a cons.
@frej frej force-pushed the frej/private-append-cleanup branch from faf517a to 1694e70 Compare February 17, 2023 11:49
@frej
Copy link
Contributor Author

frej commented Feb 17, 2023

Rebased.

@bjorng bjorng added the testing currently being tested, tag is used by OTP internal CI label Feb 17, 2023
@bjorng
Copy link
Contributor

bjorng commented Feb 17, 2023

Thanks!

@bjorng bjorng merged commit 51bc5d4 into erlang:master Feb 18, 2023
@frej frej deleted the frej/private-append-cleanup branch February 20, 2023 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants