santiontanon / mdlz80optimizer

MDL (a Z80 assembler optimizer)
Apache License 2.0
81 stars 6 forks source link

VSCode integration problem #59

Open romychs opened 1 month ago

romychs commented 1 month ago

mdl

  1. After running the task several times in a row, the same message is displayed several times. No way to cleanup list of messages, only restart VSCode
  2. For SJAsmPlus, IFUSED directive does not supported.
  3. Strange message for: SUB HL, DE instruction
ped7g commented 1 month ago

sub hl,de is fake instruction which does assemble as or a : sbc hl,de

And carry is cleared by xor a in this case, the cpir and pop de will preserve cleared carry, so you can do sbc hl,de directly. So technically mdl works as intended in this case.

theNestruo commented 1 month ago

About:

  1. After running the task several times in a row, the same message is displayed several times. No way to cleanup list of messages, only restart VSCode

Add an owner (such as: "owner": "mdl", but any value will be ok) to the problemMatcher in your .vscode/tasks.json.

It is missing in the documentation; I'll pull request the documentation change as soon as possible.

santiontanon commented 1 month ago

Thanks for the update theNestruo! I do not use VSCode, so, I didn't notice the issue! Let me know if the pull request solves the issue, and we can close it :)

theNestruo commented 1 month ago

Oh, don't close it! There is still one unattended point:

  1. For SJAsmPlus, IFUSED directive does not supported.