Amstrad CPC ASM Export Improvements

[BigWhizzy]

Amstrad CPC ASM Export Improvements

CPC-specific changes

ProcessLabelsOutsideExportedRange — bank-grouped header comments

Labels outside the exported range are now grouped by 16K block with a comment identifying the actual mapped bank and whether it contains screen memory. The original code also had a latent null-pointer bug (pLabelInfo was never checked before use) which is fixed here.

; Block 0 (0x0000-0x3fff) - ROM OS
FIRMWARE_KL_CHPR:     equ $BB5A

; Block 1 (0x4000-0x7fff) - RAM 1 (screen memory)
screen_buffer:        equ $4000

; Block 3 (0xc000-0xffff) - ROM BASIC
some_label:           equ $C000

CPCEmu::Init — ExportStartAddress/ExportEndAddress now explicitly initialised to the full address space on startup.

[Colourclash]

Thanks for taking some time to improve the cpc assembler exporter. Give me some time to look into this. My initial thoughts are the ProcessLabelsOutsideExportedRange change looks fine but I'm not sure about the bExportAsmShowAddresses change. Because it's in the shared assembler export code I would need to check with the owner of the repository. I thought about adding a similar feature in my PCE branch but then I found the .list feature of the assembler I am using (PCEAS). It outputs a listing file that lists the memory address on each line. Looks like sjasmplus has a similar feature. Would that solve your problem?

[BigWhizzy]

No worries. Yes if another tool can do the address listing then that's probably a better way. I'll rip that stuff out then. I've not yet got around to messing around sjasmplus but will have a look as well. Cheers

[BigWhizzy]

@Colourclash I have removed that option now. Cheers for the sjasmplus tip. I installed it and confirmed that I can just run sjasmplus --lst=output.lst source.asm, which produces an output.lst file with the addresses included.