You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The first one is definitely an improvement. I experimented with a few different variations of eliminating the redundant moves, and what seems to work best (after compression) is to swap the two offset writes around (inside the loop) and keeping the middle 3 instructions of the block outside the loop, so it becomes:
Hi,
I noticed some minor size opts that can be done as part of the sin-table generation
https://github.com/askeksa/Cinter/blob/master/player/Cinter4.S#L70
At this point it would be possible to do
instead as
d0
is-16384
after this loop.Also here these two pointers (+ the offsets) points to the same address so actually one write is needed.
https://github.com/askeksa/Cinter/blob/master/player/Cinter4.S#L74-L75
Now given that the code above here
https://github.com/askeksa/Cinter/blob/master/player/Cinter4.S#L63-L64 looks identical maybe the compressor will actually do a better job if it stays the same so I'm not as sure about that one.
The text was updated successfully, but these errors were encountered: