Roman Zippel - Re: Fix reload ICE with define-split (original) (raw)

This is the mail archive of the gcc-patches@gcc.gnu.orgmailing list for the GCC project.

• From: Roman Zippel
• To: Paolo Bonzini
• Cc: Nathan Sidwell , gcc-patches at gcc dot gnu dot org
• Date: Fri, 23 Feb 2007 17:45:17 +0100 (CET)
• Subject: Re: Fix reload ICE with define-split
• References: <45DEFEE5.8030905@codesourcery.com> <Pine.LNX.4.64.0702231607150.26384@scrub.home> <45DF05D0.3000800@lu.unisi.ch>

Hi,

On Fri, 23 Feb 2007, Paolo Bonzini wrote:

I have a similiar patch, which I think is a bit better. try_split() already tries to recreate notes, so the patch below adds the hook for REG_INC notes there.

This patch also has the benefit of working on dataflow branch (the problem with the other one is not insurmountable, it's just that flow.c is not present anymore on the branch).

My point is that IMO it's better to update the REG_INC note together with the other notes and not separately. I don't know anything about the dataflow branch, so I can't say how it works there.

bye, Roman

• References:
  • Fix reload ICE with define-split
    * From: Nathan Sidwell
  • Re: Fix reload ICE with define-split
    * From: Roman Zippel
  • Re: Fix reload ICE with define-split
    * From: Paolo Bonzini