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: RFA: Fix rtl-optimization/22258


On Thu, Jun 30, 2005 at 08:25:29PM +0100, Joern RENNECKE wrote:
> +   /* Disregard parts of the return value that are set later.  */
> +   for (p = PREV_INSN (use); p != insn; p = PREV_INSN (p))
> +     {
> +       set = single_set (p);
> +       if (!set || !REG_P (SET_DEST (set)))
> + 	continue;

I really don't like you creating a specialized lifetime analyzer.
Especially one that's so blatently wrong, completely ignoring
anything that's not single-set.

Even if we decide to go with a function like you suggest, you should
use one of the existing analyzers: note_stores or propagate_one_insn.


r~


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