On 03/17/2017 04:12 PM, Jim Wilson wrote:
I have access to a fast box that isn't otherwise in use at the moment so I'm taking a look. r246225 builds OK. r246226 does not. So it is Bernd's combine patch. A little experimenting shows that the compare difference is triggered by the use of -gtoggle in stage2, which is not used in stage3. Otherwise stage2 and stage3 generate identical code. The bug is apparently due to a problem with handling debug insns in the combine patch.
Changing a new prev_nonnote_insn call to a prev_nonnote_nondebug_insn call appears to solve the problem. I will have to do a bootstrap and make check from scratch to verify. I also noticed that there is a redundant i1 check in the patch which should be fixed also.
Jim