DEFINE RESET_ORG and DEC modifier issue

[richard]

i'm not really sure what exactly the code is doing but the highlighted sections are doing something with the label but ignoring the upper byte of its address. below 64k its not an issue i would not say its bug , more like its not designed to be relocated beyond the 64k boundary.
anyways its the only thing i see in the code thats likely to be an issue.

01EE52 0E04               M         movlw   low (low (HSEROUT2J))MPASM 5.61                          MCKD.ASM   1-30-2021  23:44:34         PAGE 32




LOC  OBJECT CODE     LINE SOURCE TEXT
  VALUE


01EE54 6E0A               M         movwf   R8
                          M       endif
                          M     endif
                          M         MOVE?CB high (HSEROUT2J), (R8) + 1
                          M         CHK?RP  (R8) + 1
                          M     if ((((R8) + 1) & 8000h) != PREV_ALT)
                          M       if (((R8) + 1) & 8000h)
                          M         bsf     WDTCON, ADSHR
                          M       else
                          M         bcf     WDTCON, ADSHR
                          M       endif
                          M PREV_ALT = ((R8) + 1) & 8000h
                          M     endif
                          M     if (((((R8) + 1) & 0fffh) > BANKA_END) & ((((R8) + 1) & 0fffh) < (0f01h + BANKA_END)))
                          M       if ((((R8) + 1) & 0f00h) != (PREV_BANK << 8))
                          M         movlb   high ((R8) + 1)
                          M PREV_BANK = high ((R8) + 1)
                          M       endif
                          M     endif
                          M     if (low (high (HSEROUT2J)) == 0)
                          M         clrf    (R8) + 1
                          M     else
                          M       if (low (high (HSEROUT2J)) == 255)
                          M         setf    (R8) + 1
                          M       else
01EE56 0EEC               M         movlw   low (high (HSEROUT2J))
01EE58 6E0B               M         movwf   (R8) + 1

[richard]

maybe crashing here

01EDA8 6EE9 08880 JUMPMAN movwf FSR0L
01EDAA 6AFB 08881 clrf PCLATU
01EDAC 500B 08882 movf R8 + 1, W
01EDAE 6EFA 08883 movwf PCLATH
01EDB0 500A 08884 movf R8, W
01EDB2 6EF9 08885 movwf PCL

[Marcick]

Looks like it is definitely a bug (I have bought one week ago the 3.1 update to be sure).
I'm writing in the bug report section of Melabs to see what they say. Here.
Thanks

[richard]

you should verify the 3.1 version does the same thing , i used 3.0.10
i expect it will , the space shuttle was still a thing the last time the pbp core routines were enhanced

[Marcick]

I was using 3.0 and bought 3.1 update to see if it was solved, but not.......
I think I will start to work on my subroutine to avoid use of string modifiers, though they are very powerful, it's a pity.

[richard]

be wary any pbp command that uses jumpman the same way will probably fail , there may be other avenues of failure possible too

[HenrikOlsson]

Yeah, until you here from Charles I'd be very careful with pushing the library routines beyond the 64k barrier.

I did compile the code and I did look at the .lst but I didn't see/understood the issue, nice work Richard! I know you've mostly moved on but please don't leave ;-)

/Henrik.