../../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

Reply via email to