BOOSTEDEVERYTHING wrote: ↑2024 Mar 27, 22:29
That 8f8d6 8f8de should be defined with the word command, and just because the words in between aren't directly referenced in the code, they are present so they can be indirectly referenced.
I do have one question about this though, should the word command be used in a range or should I do the addresses individually?
Indeed, indirect and indexed address modes can reference a series of addresses in a loop.
A range will get the job done quickly, so go with that.
What can be said about SAD?
Very user driven. If you do not know exactly how to use it, it will get you completely lost.. LOL
The mantra is user is king, so it will not save you from garbage in garbage out.
It does not capture the extent of loops, so the user has to be awake to that.
Do you use the branch (jump) comments I provided with RZASA?
I do, but wasn't paying attention to rzasa code when I was looking for the words referenced above.
Yeah, not so much as a comparison source for OMAE, just asking if it were useful for RZASA alone.
Would automatic SAD produced branch comments help spot the loops in subsequent code in OMAE2?
Absolutely, that makes it a ton easier to follow the code. I was not going far enough to see the loop, I may have seen it if i was following in RZASA due to the cmt file you provided me. I guess I need to try and make one for OMAE2.
I agree it is so much easier with the jump comments in place.
Generating it line by line is a long and tedious process. I wised up eventually and have a less tedious process, but really messy to do. The DIR and LST are not quite ready for that yet.
Also, I guess I need to pop over to the other thread and ask if TVRFAN could incorporate the looped scan command and possibly SAD produced branch comments, and loop comments. Also, SAD 5 still doesn't like an address to have both a word value and byte value assigned to it, but it does work in SAD 4.012....mostly.
One voice requesting this stuff does not carry as much weight as two...or more.
Yes, ask for the scan command to be updated to support range.
Yes, ask for the branch (jump) comments to be generated by SAD.
V4 ignores the duplicate byte and word.
V5 requires a size option for the sym command. I have not tested these yet, I hope they work when I get a chance, as I'd hate to bury TVRfan under more bugs right now.