Hi Jakub:
LGTM, Thanks!
On Tue, Mar 15, 2022 at 6:57 PM Jakub Jelinek via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> Hi!
>
> While checking if all targets honor -Wno-psabi for ABI related warnings
> or messages, I found that almost all do, except for riscv.
> In the testsuite when we want to ignore ABI related messages we
> typically use -Wno-psabi -w, but it would be nice to get rid of those
> -w uses eventually.
>
> The following allows silencing those warnings with -Wno-psabi rather than
> just -w even on riscv.
>
> Ok for trunk?
>
> 2022-03-15 Jakub Jelinek <ja...@redhat.com>
>
> PR target/91229
> * config/riscv/riscv.cc (riscv_pass_aggregate_in_fpr_pair_p,
> riscv_pass_aggregate_in_fpr_and_gpr_p): Pass OPT_Wpsabi instead of 0
> to warning calls.
>
> --- gcc/config/riscv/riscv.cc.jj 2022-03-07 15:00:17.239592719 +0100
> +++ gcc/config/riscv/riscv.cc 2022-03-15 11:20:37.823661044 +0100
> @@ -2918,8 +2918,8 @@ riscv_pass_aggregate_in_fpr_pair_p (cons
>
> if ((n_old != n_new) && (warned == 0))
> {
> - warning (0, "ABI for flattened struct with zero-length bit-fields "
> - "changed in GCC 10");
> + warning (OPT_Wpsabi, "ABI for flattened struct with zero-length "
> + "bit-fields changed in GCC 10");
> warned = 1;
> }
>
> @@ -2960,8 +2960,8 @@ riscv_pass_aggregate_in_fpr_and_gpr_p (c
> && (num_int_old != num_int_new || num_float_old != num_float_new)))
> && (warned == 0))
> {
> - warning (0, "ABI for flattened struct with zero-length bit-fields "
> - "changed in GCC 10");
> + warning (OPT_Wpsabi, "ABI for flattened struct with zero-length "
> + "bit-fields changed in GCC 10");
> warned = 1;
> }
>
>
> Jakub
>
