qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Cc: deller@gmx.de
Subject: [PATCH v2 08/45] target/hppa: Add install_link
Date: Mon, 13 May 2024 09:46:40 +0200	[thread overview]
Message-ID: <20240513074717.130949-9-richard.henderson@linaro.org> (raw)
In-Reply-To: <20240513074717.130949-1-richard.henderson@linaro.org>

Add a common routine for writing the return address.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/hppa/translate.c | 54 +++++++++++++++++++++++------------------
 1 file changed, 31 insertions(+), 23 deletions(-)

diff --git a/target/hppa/translate.c b/target/hppa/translate.c
index 08d5e2a4bc..f816b337ee 100644
--- a/target/hppa/translate.c
+++ b/target/hppa/translate.c
@@ -634,6 +634,23 @@ static void install_iaq_entries(DisasContext *ctx, uint64_t bi, TCGv_i64 bv,
     }
 }
 
+static void install_link(DisasContext *ctx, unsigned link, bool with_sr0)
+{
+    tcg_debug_assert(ctx->null_cond.c == TCG_COND_NEVER);
+    if (link) {
+        if (ctx->iaoq_b == -1) {
+            tcg_gen_addi_i64(cpu_gr[link], cpu_iaoq_b, 4);
+        } else {
+            tcg_gen_movi_i64(cpu_gr[link], ctx->iaoq_b + 4);
+        }
+#ifndef CONFIG_USER_ONLY
+        if (with_sr0) {
+            tcg_gen_mov_i64(cpu_sr[0], cpu_iasq_b);
+        }
+#endif
+    }
+}
+
 static inline uint64_t iaoq_dest(DisasContext *ctx, int64_t disp)
 {
     return ctx->iaoq_f + disp + 8;
@@ -1787,9 +1804,7 @@ static bool do_dbranch(DisasContext *ctx, int64_t disp,
     uint64_t dest = iaoq_dest(ctx, disp);
 
     if (ctx->null_cond.c == TCG_COND_NEVER && ctx->null_lab == NULL) {
-        if (link != 0) {
-            copy_iaoq_entry(ctx, cpu_gr[link], ctx->iaoq_n, ctx->iaoq_n_var);
-        }
+        install_link(ctx, link, false);
         ctx->iaoq_n = dest;
         if (is_n) {
             ctx->null_cond.c = TCG_COND_ALWAYS;
@@ -1797,10 +1812,7 @@ static bool do_dbranch(DisasContext *ctx, int64_t disp,
     } else {
         nullify_over(ctx);
 
-        if (link != 0) {
-            copy_iaoq_entry(ctx, cpu_gr[link], ctx->iaoq_n, ctx->iaoq_n_var);
-        }
-
+        install_link(ctx, link, false);
         if (is_n && use_nullify_skip(ctx)) {
             nullify_set(ctx, 0);
             gen_goto_tb(ctx, 0, dest, dest + 4);
@@ -1892,9 +1904,7 @@ static bool do_ibranch(DisasContext *ctx, TCGv_i64 dest,
         next = tcg_temp_new_i64();
         tcg_gen_mov_i64(next, dest);
 
-        if (link != 0) {
-            copy_iaoq_entry(ctx, cpu_gr[link], ctx->iaoq_n, ctx->iaoq_n_var);
-        }
+        install_link(ctx, link, false);
         if (is_n) {
             if (use_nullify_skip(ctx)) {
                 install_iaq_entries(ctx, -1, next, -1, NULL);
@@ -1911,16 +1921,17 @@ static bool do_ibranch(DisasContext *ctx, TCGv_i64 dest,
 
     nullify_over(ctx);
 
+    next = tcg_temp_new_i64();
+    tcg_gen_mov_i64(next, dest);
+
+    install_link(ctx, link, false);
     if (is_n && use_nullify_skip(ctx)) {
-        install_iaq_entries(ctx, -1, dest, -1, NULL);
+        install_iaq_entries(ctx, -1, next, -1, NULL);
         nullify_set(ctx, 0);
     } else {
-        install_iaq_entries(ctx, ctx->iaoq_b, cpu_iaoq_b, -1, dest);
+        install_iaq_entries(ctx, ctx->iaoq_b, cpu_iaoq_b, -1, next);
         nullify_set(ctx, is_n);
     }
-    if (link != 0) {
-        copy_iaoq_entry(ctx, cpu_gr[link], ctx->iaoq_n, ctx->iaoq_n_var);
-    }
 
     tcg_gen_lookup_and_goto_ptr();
     ctx->base.is_jmp = DISAS_NORETURN;
@@ -3899,10 +3910,7 @@ static bool trans_be(DisasContext *ctx, arg_be *a)
     nullify_over(ctx);
 
     load_spr(ctx, new_spc, a->sp);
-    if (a->l) {
-        copy_iaoq_entry(ctx, cpu_gr[31], ctx->iaoq_n, ctx->iaoq_n_var);
-        tcg_gen_mov_i64(cpu_sr[0], cpu_iasq_b);
-    }
+    install_link(ctx, a->l, true);
     if (a->n && use_nullify_skip(ctx)) {
         install_iaq_entries(ctx, -1, tmp, -1, NULL);
         tcg_gen_mov_i64(cpu_iasq_f, new_spc);
@@ -4019,16 +4027,16 @@ static bool trans_bve(DisasContext *ctx, arg_bve *a)
     return do_ibranch(ctx, dest, a->l, a->n);
 #else
     nullify_over(ctx);
-    dest = do_ibranch_priv(ctx, load_gpr(ctx, a->b));
+    dest = tcg_temp_new_i64();
+    tcg_gen_mov_i64(dest, load_gpr(ctx, a->b));
+    dest = do_ibranch_priv(ctx, dest);
 
+    install_link(ctx, a->l, false);
     install_iaq_entries(ctx, ctx->iaoq_b, cpu_iaoq_b, -1, dest);
     if (ctx->iaoq_b == -1) {
         tcg_gen_mov_i64(cpu_iasq_f, cpu_iasq_b);
     }
     tcg_gen_mov_i64(cpu_iasq_b, space_select(ctx, 0, dest));
-    if (a->l) {
-        copy_iaoq_entry(ctx, cpu_gr[a->l], ctx->iaoq_n, ctx->iaoq_n_var);
-    }
     nullify_set(ctx, a->n);
     tcg_gen_lookup_and_goto_ptr();
     ctx->base.is_jmp = DISAS_NORETURN;
-- 
2.34.1



  parent reply	other threads:[~2024-05-13  7:50 UTC|newest]

Thread overview: 100+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-13  7:46 [PATCH v2 00/45] target/hppa: Misc improvements Richard Henderson
2024-05-13  7:46 ` [PATCH v2 01/45] target/hppa: Move cpu_get_tb_cpu_state out of line Richard Henderson
2024-05-14 13:49   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 02/45] target/hppa: Use hppa_form_gva_psw in hppa_cpu_get_pc Richard Henderson
2024-05-14 14:05   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 03/45] target/hppa: Move constant destination check into use_goto_tb Richard Henderson
2024-05-14 14:06   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 04/45] target/hppa: Pass displacement to do_dbranch Richard Henderson
2024-05-14 14:07   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 05/45] target/hppa: Allow prior nullification in do_ibranch Richard Henderson
2024-05-14 14:09   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 06/45] target/hppa: Use CF_BP_PAGE instead of cpu_breakpoint_test Richard Henderson
2024-05-14 14:12   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 07/45] target/hppa: Add install_iaq_entries Richard Henderson
2024-05-14 14:23   ` Helge Deller
2024-05-13  7:46 ` Richard Henderson [this message]
2024-05-14 14:37   ` [PATCH v2 08/45] target/hppa: Add install_link Helge Deller
2024-05-14 15:43     ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 09/45] target/hppa: Delay computation of IAQ_Next Richard Henderson
2024-05-14 14:39   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 10/45] target/hppa: Skip nullified insns in unconditional dbranch path Richard Henderson
2024-05-14 14:40   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 11/45] target/hppa: Simplify TB end Richard Henderson
2024-05-14 14:43   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 12/45] target/hppa: Add IASQ entries to DisasContext Richard Henderson
2024-05-14 15:23   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 13/45] target/hppa: Add space arguments to install_iaq_entries Richard Henderson
2024-05-14 15:27   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 14/45] target/hppa: Add space argument to do_ibranch Richard Henderson
2024-05-14 15:35   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 15/45] target/hppa: Use umax in do_ibranch_priv Richard Henderson
2024-05-13 11:18   ` Philippe Mathieu-Daudé
2024-05-13 13:23     ` Richard Henderson
2024-05-13 14:15       ` Philippe Mathieu-Daudé
2024-05-13  7:46 ` [PATCH v2 16/45] target/hppa: Always make a copy " Richard Henderson
2024-05-13 10:33   ` Philippe Mathieu-Daudé
2024-05-13  7:46 ` [PATCH v2 17/45] target/hppa: Introduce and use DisasIAQE for branch management Richard Henderson
2024-05-14 15:48   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 18/45] target/hppa: Use displacements in DisasIAQE Richard Henderson
2024-05-14 15:54   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 19/45] target/hppa: Rename cond_make_* helpers Richard Henderson
2024-05-13  9:27   ` Philippe Mathieu-Daudé
2024-05-14 16:17   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 20/45] target/hppa: Use TCG_COND_TST* in do_cond Richard Henderson
2024-05-14 16:23   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 21/45] target/hppa: Use TCG_COND_TST* in do_log_cond Richard Henderson
2024-05-14 16:27   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 22/45] target/hppa: Use TCG_COND_TST* in do_unit_zero_cond Richard Henderson
2024-05-14 16:28   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 23/45] target/hppa: Use TCG_COND_TST* in do_unit_addsub Richard Henderson
2024-05-13  9:37   ` Philippe Mathieu-Daudé
2024-05-14 16:29   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 24/45] target/hppa: Use TCG_COND_TST* in trans_bb_imm Richard Henderson
2024-05-13 10:18   ` Philippe Mathieu-Daudé
2024-05-14 16:36   ` Helge Deller
2024-05-15  7:28     ` Richard Henderson
2024-05-13  7:46 ` [PATCH v2 25/45] target/hppa: Use registerfields.h for FPSR Richard Henderson
2024-05-14 18:03   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 26/45] target/hppa: Use TCG_COND_TST* in trans_ftest Richard Henderson
2024-05-13 10:27   ` Philippe Mathieu-Daudé
2024-05-14 18:05   ` Helge Deller
2024-05-13  7:46 ` [PATCH v2 27/45] target/hppa: Remove cond_free Richard Henderson
2024-05-13 10:24   ` Philippe Mathieu-Daudé
2024-05-14 18:15   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 28/45] target/hppa: Introduce DisasDelayException Richard Henderson
2024-05-14 18:49   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 29/45] target/hppa: Use delay_excp for conditional traps Richard Henderson
2024-05-14 18:55   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 30/45] target/hppa: Use delay_excp for conditional trap on overflow Richard Henderson
2024-05-14 19:00   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 31/45] linux-user/hppa: Force all code addresses to PRIV_USER Richard Henderson
2024-05-14 19:58   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 32/45] target/hppa: Store full iaoq_f and page offset of iaoq_b in TB Richard Henderson
2024-05-14 20:11   ` Helge Deller
2024-05-15  7:35     ` Richard Henderson
2024-05-13  7:47 ` [PATCH v2 33/45] target/hppa: Do not mask in copy_iaoq_entry Richard Henderson
2024-05-14 20:13   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 34/45] target/hppa: Improve hppa_cpu_dump_state Richard Henderson
2024-05-14 20:16   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 35/45] target/hppa: Split PSW X and B into their own field Richard Henderson
2024-05-13 10:30   ` Philippe Mathieu-Daudé
2024-05-14 20:22   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 36/45] target/hppa: Manage PSW_X and PSW_B in translator Richard Henderson
2024-05-14 21:18   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 37/45] target/hppa: Implement PSW_B Richard Henderson
2024-05-14 21:19   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 38/45] target/hppa: Implement PSW_X Richard Henderson
2024-05-14 21:22   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 39/45] target/hppa: Drop tlb_entry return from hppa_get_physical_address Richard Henderson
2024-05-13  9:26   ` Philippe Mathieu-Daudé
2024-05-14 21:23   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 40/45] target/hppa: Adjust priv for B,GATE at runtime Richard Henderson
2024-05-14 21:26   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 41/45] target/hppa: Implement CF_PCREL Richard Henderson
2024-05-14 21:32   ` Helge Deller
2024-05-13  7:47 ` [PATCH v2 42/45] target/hppa: Implement PSW_T Richard Henderson
2024-05-13  7:47 ` [PATCH v2 43/45] target/hppa: Implement PSW_H, PSW_L Richard Henderson
2024-05-13  7:47 ` [PATCH v2 44/45] target/hppa: Log cpu state at interrupt Richard Henderson
2024-05-13  7:47 ` [PATCH v2 45/45] target/hppa: Log cpu state on return-from-interrupt Richard Henderson
2024-05-14 14:02 ` [PATCH v2 00/45] target/hppa: Misc improvements Helge Deller

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=20240513074717.130949-9-richard.henderson@linaro.org \
    --to=richard.henderson@linaro.org \
    --cc=deller@gmx.de \
    --cc=qemu-devel@nongnu.org \
    /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 NNTP newsgroup(s).