https://gcc.gnu.org/bugzilla/show_bug.cgi?id=127934

Drea Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2026-10-16
             Status|UNCONFIRMED                 |NEW

--- Comment #1 from Drea Pinski <pinskia at gcc dot gnu.org> ---
Review of the patch, instead of using next_nonnote_nondebug_insn, I think
adding/creating get_first_nonnote_nondebug_insn would be better.

The two places that use get_first_nonnote_insn should be moved over to the new
function too.
That is the one in riscv too:
static int
riscv_mode_needed (int entity, rtx_insn *insn, HARD_REG_SET)
{
  int code = recog_memoized (insn);

  switch (entity)
    {
    case RISCV_VXRM:
      /* If CUR_INSN is the first insn in the function, then determine if we
         want to signal a need in ENTRY->succs to allow for aggressive
         elimination of subsequent sets of VXRM.  */
      if (insn == get_first_nonnote_insn ())
        {
          int need = singleton_vxrm_need ();
          if (need != VXRM_MODE_NONE)
            return need;
        }

Reply via email to