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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch RFC] Fix bogus RTL sharing error


Kaz Kojima <kkojima@rr.iij4u.or.jp> writes:

> Ian Lance Taylor <iant@google.com> wrote:
> > I was thinking that you would handle them in reset_used_flags since
> > that seemed simpler.  But they don't have to be handled recursively.
> > Whatever seems right to you.
> 
> I agreed that changing reset_used_flags will be simpler, but
> I'm afraid that it changes the semantic of reset_used_flags
> slightly and breaks the symmetry for set_used_frags.
> Could you approve the patch in
> http://gcc.gnu.org/ml/gcc-patches/2006-11/msg01935.html
> for the mainline if it passes the usual tests on i686-linux?

OK, but don't test INSN_P (q).  It should always be true.  You can
make it a gcc_assert if you like.

Thanks.

Ian


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]