Skip to content
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

PIOasm disasm comment wrong #1961

Open
NollKollTroll opened this issue Sep 29, 2024 · 0 comments
Open

PIOasm disasm comment wrong #1961

NollKollTroll opened this issue Sep 29, 2024 · 0 comments
Assignees
Milestone

Comments

@NollKollTroll
Copy link

NollKollTroll commented Sep 29, 2024

When compiling the following test-code:

.program test
.fifo txput
.wrap_target
    mov  rxfifo[0], ISR
    mov  rxfifo[1], ISR
    mov  rxfifo[2], ISR
    mov  rxfifo[3], ISR
    mov  rxfifo[Y], ISR
.wrap

I seem to get a correct binary according to the documentation. Sadly the disasm comments are wrong, swapping indexed and Y-mode:

static const uint16_t test_program_instructions[] = {
            //     .wrap_target
    0x8018, //  0: mov    rxfifo[y], isr             
    0x8019, //  1: mov    rxfifo[y], isr             
    0x801a, //  2: mov    rxfifo[y], isr             
    0x801b, //  3: mov    rxfifo[y], isr             
    0x8010, //  4: mov    rxfifo[0], isr             
            //     .wrap
};

I tried very hard to use code-blocks in this text, but the line-endings failed hard in the preview whatever I tried.

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

No branches or pull requests

2 participants