public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: binutils@sourceware.org
Cc: gdb-patches@sourceware.org, shorne@gmail.com, blue@cmd.nu,
	stefan.kristiansson@saunalahti.fi
Subject: Re: [PATCH] cpu: or1k: drop unused l.swa flag
Date: Wed, 20 Dec 2023 22:26:13 -0500	[thread overview]
Message-ID: <ZYOwVaXYL2hKY-Dj@vapier> (raw)
In-Reply-To: <20231219023003.13985-1-vapier@gentoo.org>

[-- Attachment #1: Type: text/plain, Size: 1933 bytes --]

add some or1k maintainers ...
-mike

On 18 Dec 2023 21:30, Mike Frysinger wrote:
> The "flag" argument isn't set/used in this insn, so drop it.
> This fixes an unused variable warning in the generated sim.
> ---
>  cpu/or1korbis.cpu     | 2 +-
>  sim/or1k/sem-switch.c | 1 -
>  sim/or1k/sem.c        | 1 -
>  3 files changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/cpu/or1korbis.cpu b/cpu/or1korbis.cpu
> index a8002a371858..b4ecb84ab413 100644
> --- a/cpu/or1korbis.cpu
> +++ b/cpu/or1korbis.cpu
> @@ -717,7 +717,7 @@
>       ((MACH ORBIS-MACHS))
>       "l.swa ${simm16-split}($rA),$rB"
>       (+ OPC_SWA rA rB simm16)
> -     (sequence ((SI addr) (BI flag))
> +     (sequence ((SI addr))
>  	       (set addr (load-store-addr rA simm16-split 4))
>  	       (set sys-sr-f (and atomic-reserve (eq addr atomic-address)))
>  	       (if sys-sr-f
> diff --git a/sim/or1k/sem-switch.c b/sim/or1k/sem-switch.c
> index b63add8817b7..dccc65a0ec14 100644
> --- a/sim/or1k/sem-switch.c
> +++ b/sim/or1k/sem-switch.c
> @@ -1056,7 +1056,6 @@ if (EQSI (ANDSI (tmp_addr, 268435452), CPU (h_atomic_address))) {
>  
>  {
>    SI tmp_addr;
> -  BI tmp_flag;
>    tmp_addr = or1k32bf_make_load_store_addr (current_cpu, GET_H_GPR (FLD (f_r2)), EXTSISI (FLD (f_simm16_split)), 4);
>    {
>      USI opval = ANDBI (CPU (h_atomic_reserve), EQSI (tmp_addr, CPU (h_atomic_address)));
> diff --git a/sim/or1k/sem.c b/sim/or1k/sem.c
> index 95c474dc478b..973590d41154 100644
> --- a/sim/or1k/sem.c
> +++ b/sim/or1k/sem.c
> @@ -933,7 +933,6 @@ SEM_FN_NAME (or1k32bf,l_swa) (SIM_CPU *current_cpu, SEM_ARG sem_arg)
>  
>  {
>    SI tmp_addr;
> -  BI tmp_flag;
>    tmp_addr = or1k32bf_make_load_store_addr (current_cpu, GET_H_GPR (FLD (f_r2)), EXTSISI (FLD (f_simm16_split)), 4);
>    {
>      USI opval = ANDBI (CPU (h_atomic_reserve), EQSI (tmp_addr, CPU (h_atomic_address)));
> -- 
> 2.43.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      reply	other threads:[~2023-12-21  3:26 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-19  2:30 Mike Frysinger
2023-12-21  3:26 ` Mike Frysinger [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZYOwVaXYL2hKY-Dj@vapier \
    --to=vapier@gentoo.org \
    --cc=binutils@sourceware.org \
    --cc=blue@cmd.nu \
    --cc=gdb-patches@sourceware.org \
    --cc=shorne@gmail.com \
    --cc=stefan.kristiansson@saunalahti.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).