Re: Problem w/PIC and C++ exceptions in 2.95 on IA-32

[Clinton Popetz <cpopetz at cygnus dot com>]

Franz Sirl <Franz.Sirl-kernel@lauterbach.com> writes:

> >  > This only matters if we plan on issuing 2.95.3.  Do we have sufficient
> >  > minor bugs to justify that?
> >We should seriously consider it given the number of bugfixes I've pulled
> >into that branch "just in case" and the fact that we haven't set a release
> >schedule for the next major release.
> 
> I agree, though I have some suggestions for additional patches to backport:
> 
> - Clinton Popetz recent one-liner loop fix
> 2000-01-18  Clinton Popetz  <cpopetz@cygnus.com>
>  
>         * loop.c (check_dbra_loop): When checking a loop for
>         reversability, check the source of any stores to ensure they
>         don't depend on an initial value.                                                                                              

Actually, this one was never approved for the mainline; I think it was
missing a CC to gcc-patches, as I can't find it in the archives.
The patch is at:

	http://gcc.gnu.org/ml/gcc-bugs/2000-01/msg00463.html

and is appended here for reference.

				-Clint

===================================================================
RCS file: /cvs/gcc/egcs/gcc/loop.c,v
retrieving revision 1.217
diff -c -2 -p -r1.217 loop.c
*** loop.c      2000/01/15 03:01:49     1.217
--- loop.c      2000/01/18 06:53:21
*************** check_dbra_loop (loop, insn_count)
*** 8035,8039 ****
                  if (v->giv_type == DEST_REG
                      && reg_mentioned_p (v->dest_reg,
!                                         XEXP (loop_store_mems, 0))
                      && loop_insn_first_p (first_loop_store_insn, v->insn))
                    reversible_mem_store = 0;
--- 8035,8039 ----
                  if (v->giv_type == DEST_REG
                      && reg_mentioned_p (v->dest_reg,
!                                        PATTERN (first_loop_store_insn)) 
                      && loop_insn_first_p (first_loop_store_insn, v->insn))
                    reversible_mem_store = 0;