When adding a ChangeLog entry for a recent change I realized the define_insn_and_split didn't have a name. So I added the name, but didn't account for the iterator and thus we have the same name appearing multiple times.

Anyway, this just adds the '*' prefix so that the name is considered for debugging purposes only.

Pushing as obvious.

Jeff

ps.  Yes, you could argue that I should have written the ChangeLog when I wrote the patch.
commit a1c59582e02e8bc56b5623230c5b26508e745bec
Author: Jeff Law <[email protected]>
Date:   Thu Jun 25 09:37:13 2026 -0600

    [RISC-V] Avoid multiply defined insn name
    
    When adding a ChangeLog entry for a recent change I realized the
    define_insn_and_split didn't have a name.  So I added the name, but didn't
    account for the iterator and thus we have the same name appearing multiple
    times.
    
    Anyway, this just adds the '*' prefix so that the name is considered for
    debugging purposes only.
    
    Pushing as obvious.
    
    Jeff
    
    ps.  Yes, you could argue that I should have written the ChangeLog when I 
wrote
    the patch.
    
    gcc/
            * config/riscv/riscv.md (seq_sne_qi): Mark name for debugging 
purposes
            only.

diff --git a/gcc/config/riscv/riscv.md b/gcc/config/riscv/riscv.md
index d0cfa2c81ed..2cd26fa99ab 100644
--- a/gcc/config/riscv/riscv.md
+++ b/gcc/config/riscv/riscv.md
@@ -5363,7 +5363,7 @@ (define_split
 ;;
 ;; There's probably some HImode cases we could handle too.  I haven't
 ;; thought hard about them.
-(define_insn_and_split "seq_sne_qi"
+(define_insn_and_split "*seq_sne_qi"
   [(set (match_operand:X 0 "register_operand" "=r")
        (any_eq:X (subreg:QI
                   (ashift:X (match_operand:X 1 "register_operand" "r")

Reply via email to