Skip to content

OP_APPLY_LAST: fix stacktrace on failure #1758

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
Jul 21, 2025

Conversation

pguyot
Copy link
Collaborator

@pguyot pguyot commented Jul 15, 2025

Truncate stack only after checking module and function, thus generating a more helpful stacktrace, following what BEAM does.

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

@pguyot pguyot changed the base branch from main to release-0.6 July 15, 2025 20:58
@pguyot pguyot force-pushed the w29/fix-apply-last-stacktrace branch from 78257dd to 0663af4 Compare July 15, 2025 20:59
@pguyot pguyot force-pushed the w29/fix-apply-last-stacktrace branch 2 times, most recently from fe67faf to 536cc3c Compare July 16, 2025 04:43
@pguyot
Copy link
Collaborator Author

pguyot commented Jul 16, 2025

There is a 32 bit issue that needs to be fixed

The issue was related to AVM_CREATE_STACKTRACES=off

@pguyot pguyot marked this pull request as draft July 16, 2025 16:15
Truncate stack only after checking module and function, thus generating a
more helpful stacktrace, following what BEAM does.

Signed-off-by: Paul Guyot <[email protected]>
@pguyot pguyot force-pushed the w29/fix-apply-last-stacktrace branch from 536cc3c to 2b95eb4 Compare July 19, 2025 20:58
@pguyot pguyot marked this pull request as ready for review July 19, 2025 21:46
@bettio bettio merged commit c462803 into atomvm:release-0.6 Jul 21, 2025
136 of 142 checks passed
@pguyot pguyot deleted the w29/fix-apply-last-stacktrace branch July 21, 2025 21:46
bettio added a commit that referenced this pull request Jul 22, 2025
Merge fixes for bnot support to boxed integers (#1743) and stacktrace
with OP_APPLY_LAST (#1758).
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.

2 participants