../../gcc/config/tilegx/tilegx.md:4109:1: ambiguous attribute 'n'; could be '1' (via 'I124MODE:n') or '4' (via 'I48MODE:n') ../../gcc/config/tilegx/tilegx.md:4109:1: ambiguous attribute 'n'; could be '1' (via 'I124MODE:n') or '' (via 'I48MODE:n') ../../gcc/config/tilegx/tilegx.md:4109:1: ambiguous attribute 'n'; could be '2' (via 'I124MODE:n') or '4' (via 'I48MODE:n') ../../gcc/config/tilegx/tilegx.md:4109:1: ambiguous attribute 'n'; could be '2' (via 'I124MODE:n') or '' (via 'I48MODE:n') ../../gcc/config/tilegx/tilegx.md:4109:1: ambiguous attribute 'n'; could be '4' (via 'I124MODE:n') or '' (via 'I48MODE:n')
The insn name already uses <I124MODE:n> explicitly, just the preparation stmts don't, and as it creates a I124MODE lowpart subreg of a word mode register, <I124MODE:n> seems obviously correct. Committed as obvious. 2020-05-02 Jakub Jelinek <ja...@redhat.com> * config/tilegx/tilegx.md (insn_stnt<I124MODE:n>_add<I48MODE:bitsuffix>): Use <I124MODE:n> rather than just <n>. --- gcc/config/tilegx/tilegx.md.jj 2020-01-12 11:54:36.431413137 +0100 +++ gcc/config/tilegx/tilegx.md 2020-05-02 12:04:14.092322428 +0200 @@ -4110,7 +4110,7 @@ (define_expand "insn_stnt<I124MODE:n>_ad operands[1] = simplify_gen_subreg (<I124MODE:MODE>mode, operands[1], DImode, BYTES_BIG_ENDIAN - ? UNITS_PER_WORD - <n> : 0); + ? UNITS_PER_WORD - <I124MODE:n> : 0); }) (define_insn "*insn_stnt<I124MODE:n>_add<I48MODE:bitsuffix>" Jakub