This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: question about target info cache in resource.c
- From: Ian Lance Taylor <iant at google dot com>
- To: Tom de Vries <vries at codesourcery dot com>
- Cc: gcc at gcc dot gnu dot org, Maxim Kuvyrkov <maxim at codesourcery dot com>, Nathan Sidwell <nathan at codesourcery dot com>
- Date: Tue, 11 Jan 2011 06:28:24 -0800
- Subject: Re: question about target info cache in resource.c
- References: <4D2C29C9.5010506@codesourcery.com>
Tom de Vries <vries@codesourcery.com> writes:
> The following fix makes sure that the cached live info is invalidated:
> ...
> @@ -1865,6 +1880,7 @@ redundant_insn (rtx insn, rtx target, rt
> {
> /* Show that this insn will be used in the sequel. */
> INSN_FROM_TARGET_P (candidate) = 0;
> + incr_ticks_for_insn (candidate);
> return candidate;
> }
>
> ...
> and the assert is not triggered anymore.
>
> So my questions are:
> - is the consistency check correct? Does it make sense to fix all the
> cases where it triggers?
> - Is my analysis of the example and the fix correct?
Sounds reasonable to me.
Ian