vhelin / wla-dx

WLA DX - Yet Another GB-Z80/Z80/Z80N/6502/65C02/65CE02/65816/68000/6800/6801/6809/8008/8080/HUC6280/SPC-700/SuperFX Multi Platform Cross Assembler Package
Other
549 stars 98 forks source link

Weird error with known-good code #640

Closed Ramsis-SNES closed 3 weeks ago

Ramsis-SNES commented 1 month ago

Hi,

I'd like to update an older project of mine, just some housekeeping basically. ;-)

However, when trying to assemble it using the latest WLA sources (or v10.6 for that matter), I'm getting a strange error:

wla-65816 -o ct_msu1.o ct_msu1.asm ct_msu1.asm:96: ERROR: Cannot process "phd". Syntax error?

If I comment out that line, it'll complain about another stack-related instruction later on. :-(

Feel free to try it out, the archive contains the bare minimum of files required:

ct_msu_test.zip

You still need an original Chrono Trigger ROM (named ct.sfc) as a .BACKGROUND though.

Thanks for any help!

vhelin commented 4 weeks ago

Thanks for the report! I'm not sure if I can dedicate time to this tomorrow, but hopefully on Wednesday!

vhelin commented 3 weeks ago

Hi! I tried

kuva

... and it worked without errors:

kuva

Have you done "make clean" and then "make" for WLA DX?

vhelin commented 3 weeks ago

I'm using the very latest sources here, v10.7a, but I don't see anything on CHANGELOG that would affect this case since v10.6...

Ramsis-SNES commented 3 weeks ago

Have you done "make clean" and then "make" for WLA DX?

Ugh, I didn't know that would make a difference. Thanks a lot, Ville, I tried that and it works now. :-)

vhelin commented 3 weeks ago

Have you done "make clean" and then "make" for WLA DX?

Ugh, I didn't know that would make a difference. Thanks a lot, Ville, I tried that and it works now. :-)

Great that it worked! :) If there are changes in the updated sources to instructions table(s), then one needs to do "make clean" + "make" to get working executables, otherwise there will be lots of instruction parsing related issues.

Ramsis-SNES commented 3 weeks ago

Ah, that explains it. I'll definitely keep that in mind. :-)