From: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
To: qemu-devel@nongnu.org
Cc: qemu-riscv@nongnu.org, alistair.francis@wdc.com,
bmeng@tinylab.org, liwei1518@gmail.com,
zhiwei_liu@linux.alibaba.com, palmer@rivosinc.com,
richard.henderson@linaro.org, philmd@linaro.org,
Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Subject: [PATCH for-9.0 v14 0/8] riscv: set vstart_eq_zero on vector insns
Date: Wed, 13 Mar 2024 19:01:33 -0300 [thread overview]
Message-ID: <20240313220141.427730-1-dbarboza@ventanamicro.com> (raw)
Hi,
In this version we're fixing a redundant check in the vmvr_v helper that
was pointed out by in v13.
To make this change easier patches 3 and 4 switched places. A trivial
change was made in patch 4 that don't warrant another review.
We're missing acks in patch 3 only.
Series based on master.
Changes from v13:
- patches 3 and 4: switched places
- patch 3:
- fixed commit msg: from
"(...) now sure that vstart is being clearer"
to
"(...) now sure that vstart is being cleared"
- patch 4:
- do not check for vstart >= vl (i.e. add VSTART_CHECK_EARLY_EXIT())
in the 'vmvr_v' helper
- v13 link: https://lore.kernel.org/qemu-riscv/20240313193059.405329-1-dbarboza@ventanamicro.com/
Daniel Henrique Barboza (7):
target/riscv/vector_helper.c: set vstart = 0 in GEN_VEXT_VSLIDEUP_VX()
trans_rvv.c.inc: set vstart = 0 in int scalar move insns
target/riscv: always clear vstart in whole vec move insns
target/riscv/vector_helpers: do early exit when vstart >= vl
target/riscv: remove 'over' brconds from vector trans
trans_rvv.c.inc: remove redundant mark_vs_dirty() calls
target/riscv/vector_helper.c: optimize loops in ldst helpers
Ivan Klokov (1):
target/riscv: enable 'vstart_eq_zero' in the end of insns
target/riscv/insn_trans/trans_rvbf16.c.inc | 18 +-
target/riscv/insn_trans/trans_rvv.c.inc | 198 +++++----------------
target/riscv/insn_trans/trans_rvvk.c.inc | 30 +---
target/riscv/translate.c | 6 +
target/riscv/vcrypto_helper.c | 32 ++++
target/riscv/vector_helper.c | 100 ++++++++++-
target/riscv/vector_internals.c | 4 +
target/riscv/vector_internals.h | 9 +
8 files changed, 205 insertions(+), 192 deletions(-)
--
2.43.2
next reply other threads:[~2024-03-13 22:02 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-13 22:01 Daniel Henrique Barboza [this message]
2024-03-13 22:01 ` [PATCH for-9.0 v14 1/8] target/riscv/vector_helper.c: set vstart = 0 in GEN_VEXT_VSLIDEUP_VX() Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 2/8] trans_rvv.c.inc: set vstart = 0 in int scalar move insns Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 3/8] target/riscv: always clear vstart in whole vec " Daniel Henrique Barboza
2024-03-13 22:36 ` Richard Henderson
2024-03-14 9:43 ` Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 4/8] target/riscv/vector_helpers: do early exit when vstart >= vl Daniel Henrique Barboza
2024-03-14 3:52 ` Max Chou
2024-03-14 9:47 ` Daniel Henrique Barboza
2024-03-14 13:14 ` Max Chou
2024-03-14 13:27 ` Daniel Henrique Barboza
2024-03-14 13:42 ` Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 5/8] target/riscv: remove 'over' brconds from vector trans Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 6/8] trans_rvv.c.inc: remove redundant mark_vs_dirty() calls Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 7/8] target/riscv: enable 'vstart_eq_zero' in the end of insns Daniel Henrique Barboza
2024-03-13 22:01 ` [PATCH for-9.0 v14 8/8] target/riscv/vector_helper.c: optimize loops in ldst helpers Daniel Henrique Barboza
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=20240313220141.427730-1-dbarboza@ventanamicro.com \
--to=dbarboza@ventanamicro.com \
--cc=alistair.francis@wdc.com \
--cc=bmeng@tinylab.org \
--cc=liwei1518@gmail.com \
--cc=palmer@rivosinc.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-riscv@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=zhiwei_liu@linux.alibaba.com \
/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).