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")