On 03/17/2017 05:51 PM, Jim Wilson wrote:
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.
Thanks Jim. My 246226 was just spinning up.
Consider your fixes pre-approved.
jeff
