> > I fixed this (in patch 5) by introducing a new local rtx "set" for the > > result of single_set, and hence not overwriting "insn" within the loop. > > That said I've only tested that it compiles for rl78, I've not yet > > forced line 3605 to execute, and not simulated the resulting code. > Seems reasonable. DJ might want to chime in here with a comment or two > for testing, but I'm comfortable with how you've addressed this issue.
Given the amount of dead sets in rl78's code, I think you'd have a hard time *not* executing that line ;-) But yes, your interpretation of what should have been happening is correct. Thanks! Given that this type of bug is what your changes are *supposed* to catch, I think you should claim success now!