3.0.1 Freeze

Richard Henderson rth@redhat.com
Wed Aug 8 12:56:00 GMT 2001


On Wed, Aug 08, 2001 at 09:17:00PM +0200, Roman Zippel wrote:
> So the correct patch should possibly look like below, which also had the
> advantage to handle multiple sets.

I don't want to get into handling multiple sets.

I think something like

        set = single_set (insn);
        if (! set)
          continue;

        /* Note that single_set ignores parts of a parallel set for
           which one of the destinations is REG_UNUSED.  We can't
           handle that here, since we can wind up rewriting things
           such that a single register is set twice within a single
           parallel.  */
        if (reg_set_p (src, insn))
          continue;
           
would be a lot safer.  And could go on the 3.0 branch without ifdefs,
since it does nothing but disable optimization for particular instances.


r~



More information about the Gcc mailing list