Skip to content

Conversation

@pguyot
Copy link
Collaborator

@pguyot pguyot commented Nov 29, 2022

Add support for the following two opcodes, generated by OTP compiler in current master branch:

  • update_record/5
  • bs_match/3

The opcodes were found in many existing tests, however test_set_tuple_element was updated to cover the hints of
update_record/5.

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 title Add OTP26 opcodes Add support for otp26 opcodes Nov 29, 2022
@pguyot pguyot force-pushed the w48/add-otp26-opcodes branch 3 times, most recently from 1a17e9e to fec8dc5 Compare December 1, 2022 07:57
@pguyot pguyot force-pushed the w48/add-otp26-opcodes branch from fec8dc5 to e01d9d7 Compare January 7, 2023 14:28
@pguyot pguyot marked this pull request as ready for review January 7, 2023 14:30
@pguyot pguyot force-pushed the w48/add-otp26-opcodes branch from e01d9d7 to 7a9204f Compare January 7, 2023 14:44
@pguyot pguyot force-pushed the w48/add-otp26-opcodes branch from 7a9204f to ac2149c Compare January 7, 2023 17:04
pguyot added 2 commits January 7, 2023 18:15
Add support for the following two opcodes, generated by OTP compiler in
current master branch:
- `update_record/5`
- `bs_match/3`

The opcodes were found in many existing tests, however
`test_set_tuple_element` was updated to cover the hints of
`update_record/5`.

Signed-off-by: Paul Guyot <pguyot@kallisys.net>
Signed-off-by: Paul Guyot <pguyot@kallisys.net>
@pguyot pguyot force-pushed the w48/add-otp26-opcodes branch from ac2149c to 05e3809 Compare January 7, 2023 17:15
@bettio bettio merged commit b592431 into atomvm:master Jan 7, 2023
@pguyot pguyot deleted the w48/add-otp26-opcodes branch January 7, 2023 19:18
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