On Mon, Feb 24, 2025 at 10:46 AM Richard Biener
<richard.guent...@gmail.com> wrote:

>           /* Otherwise, if this register is used by I3, then this register
>              now dies here, so we must put a REG_DEAD note here unless there
>              is one already.  */
>           else if (reg_referenced_p (XEXP (note, 0), PATTERN (i3))
>              {
>                  if (! (REG_P (XEXP (note, 0))
>                          ? find_regno_note (i3, REG_DEAD,
>                                             REGNO (XEXP (note, 0)))
>                          : find_reg_note (i3, REG_DEAD, XEXP (note, 0))))
>                {
>                  PUT_REG_NOTE_KIND (note, REG_DEAD);
>                  place = i3;
>                }
>              }
>
> ?  At least the else { case seems to assume the reg isn't refernced in i3.
> The comment wording might also need an update of course.

Reading the comment a couple of times, considering that "here" and
"there" means I3 and that "one" means REG_DEAD note for "this
register" in I3, the comment actually makes sense.

Uros.

Reply via email to