Menu

#3851 mos6502 port symbols

closed-fixed
nobody
None
MOS6502
5
2026-05-17
2025-05-25
No

The mos6502 port now uses some symbols that are neither static, nor contain the port name: at least addSign aopOp, freeAsmop.
All port-specific symbols should be either static (like aopOp in src/z80/gen.c), or be prefixed with the port name (like z80IsReturned in src/z80/gen.c). Otherwise we get problems when linking SDCC (as soon as two ports do what they shouldn't - in this case both the mos6502 port and the pic14 port use these symbols).

P.S.: I'll fix the issue for these three function on the pic14 side. This bug report is about the issue on the mos6502 side.

Related

Bugs: #3853

Discussion

  • Philipp Klaus Krause

    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,2 +1,4 @@
    -The mos6502 port now uses some symbols that are neither static, nor contain the port name: addSign aopOp, freeAsmop.
    +The mos6502 port now uses some symbols that are neither static, nor contain the port name: at least addSign aopOp, freeAsmop.
     All port-specific symbols should be either static (like aopOp in src/z80/gen.c), or be prefixed with the port name (like z80IsReturned in src/z80/gen.c). Otherwise we get problems when linking SDCC (as soon as two ports do what they shouldn't - in this case both the mos6502 port and the pic14 port use these symbols).
    +
    +P.S.: I'll fix the issue for these three function on the pic14 side. This bug report is about the issue on the mos6502 side.
    
     
  • Philipp Klaus Krause

    As of [r15449], we should have a working fix on the pic14 side, It is now possible again to build SDCC that supports both pic14 and mos6502 (and this is now the default again). @gorlik: Still, we should also fix the mos6502 side.

     

    Related

    Commit: [r15449]

  • Gabriele Gorla

    Gabriele Gorla - 2026-03-14

    I am slowly adding the m6502_ prefix to all external symbols. [r16357]

     

    Related

    Commit: [r16357]

  • Gabriele Gorla

    Gabriele Gorla - 2026-05-17

    fixed in [r16536]

     

    Related

    Commit: [r16536]

  • Gabriele Gorla

    Gabriele Gorla - 2026-05-17
    • status: open --> closed-fixed
     

Log in to post a comment.

Monday.com Logo