This code is mostly the same, except that there's no write to memory to store intj and reload. However, this is bad codegen, as the negative flag is set based on the byte value written, not the full word in D0.
I don't yet know when this bug was introduced into the compiler, but I'm going to get it fixed ASAP, and the fix will be in 9.3. Thanks for the report!
A quick followup -- the bug was introduced during another bug fix. The peephole optimization that lets the compiler remove the tst instruction in some situations didn't have a check that the preceding move and the test were using the same sized operands. I've committed a fix to our version control system, and this will be in 9.3.
--
Ben Combee <[EMAIL PROTECTED]>
CodeWarrior for Palm OS technical lead
Palm OS programming help @ www.palmoswerks.com
-- For information on using the Palm Developer Forums, or to unsubscribe, please see http://www.palmos.com/dev/support/forums/
