[PATCH 2/2] RISC-V: Fix inconsistent error message (range)

Nelson Chu nelson.chu@sifive.com
Wed Jun 22 10:36:01 GMT 2022


LGTM, so committed

Thanks
Nelson

On Fri, Jun 10, 2022 at 6:00 PM Tsukasa OI <research_trasio@irq.a4lg.com> wrote:
>
> This commit fixes inconsistent error message format involving compressed
> funct<n> fields.  In specific, funct6 had an error message with range
> 0..2^<n> ("0..64") unlike other funct<n> fields with 0..2^<n>-1
> (e.g. funct4 with "0..15").
>
> gas/ChangeLog:
>
>         * config/tc-riscv.c (riscv_ip): Fix inconsistent error message.
> ---
>  gas/config/tc-riscv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gas/config/tc-riscv.c b/gas/config/tc-riscv.c
> index 1b730b4be36..a10ae8e237b 100644
> --- a/gas/config/tc-riscv.c
> +++ b/gas/config/tc-riscv.c
> @@ -2582,7 +2582,7 @@ riscv_ip (char *str, struct riscv_cl_insn *ip, expressionS *imm_expr,
>                             || imm_expr->X_add_number >= 64)
>                           {
>                             as_bad (_("bad value for compressed funct6 "
> -                                     "field, value must be 0...64"));
> +                                     "field, value must be 0...63"));
>                             break;
>                           }
>                         INSERT_OPERAND (CFUNCT6, *ip, imm_expr->X_add_number);
> --
> 2.34.1
>


More information about the Binutils mailing list