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.