From: Philipp Tomsich <philipp.tomsich@vrull.eu>
To: Kito Cheng <kito.cheng@gmail.com>
Cc: Christoph Muellner <christoph.muellner@vrull.eu>,
gcc-patches@gcc.gnu.org, Kito Cheng <kito.cheng@sifive.com>,
Jim Wilson <jim.wilson.gcc@gmail.com>,
Palmer Dabbelt <palmer@dabbelt.com>,
Andrew Waterman <andrew@sifive.com>,
Cooper Qu <cooper.qu@linux.alibaba.com>,
Lifang Xia <lifang_xia@linux.alibaba.com>,
Yunhai Shang <yunhai@linux.alibaba.com>,
Zhiwei Liu <zhiwei_liu@linux.alibaba.com>,
Andrew Pinski <pinskia@gmail.com>,
Hans-Peter Nilsson <hp@bitrange.com>
Subject: Re: [PATCH v4 0/9] RISC-V: Add XThead* extension support
Date: Wed, 15 Mar 2023 10:02:06 +0100 [thread overview]
Message-ID: <CAAeLtUDyqj6gXmu=0Qp2W2jMaokCGs_Ff3kCX5S-Jj6aQEyR9A@mail.gmail.com> (raw)
In-Reply-To: <CA+yXCZBtv=WeNUxR_Pr-d+=e9pqp5BAKvn-e6VgeH1JAncfwXA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 8686 bytes --]
On Sun, 5 Mar 2023 at 11:19, Kito Cheng <kito.cheng@gmail.com> wrote:
> LGTM :)
>
Applied to master, thanks!
--Philipp.
On Thu, Mar 2, 2023 at 4:36 PM Christoph Muellner
> <christoph.muellner@vrull.eu> wrote:
> >
> > From: Christoph Müllner <christoph.muellner@vrull.eu>
> >
> > This series introduces support for the T-Head specific RISC-V ISA
> extensions
> > which are available e.g. on the T-Head XuanTie C906.
> >
> > The ISA spec can be found here:
> > https://github.com/T-head-Semi/thead-extension-spec
> >
> > This series adds support for the following XThead* extensions:
> > * XTheadBa
> > * XTheadBb
> > * XTheadBs
> > * XTheadCmo
> > * XTheadCondMov
> > * XTheadFmv
> > * XTheadInt
> > * XTheadMac
> > * XTheadMemPair
> > * XTheadSync
> >
> > All extensions are properly integrated and the included tests
> > demonstrate the improvements of the generated code.
> >
> > The series also introduces support for "-mcpu=thead-c906", which also
> > enables all available XThead* ISA extensions of the T-Head C906.
> >
> > All patches have been tested and don't introduce regressions for RV32 or
> RV64.
> > The patches have also been tested with SPEC CPU2017 on QEMU and real HW
> > (D1 board).
> >
> > Support patches for these extensions for Binutils, QEMU, and LLVM have
> > already been merged in the corresponding upstream projects.
> >
> > Patches 1-8 from this series (everything except the last one) got an ACK
> > by Kito. However, since there were a few comments after the ACK, I
> > decided to send out a v4, so that reviewers can verify that their
> > comments have been addressed properly.
> >
> > Note, that there was a concern raised by Andrew Pinski (on CC), which
> > might not be resolved with this series (I could not reproduce the issue,
> > but I might have misunderstood something).
> >
> > Changes in v4:
> > - Drop XTheadMemIdx and XTheadFMemIdx (will be a follow-up series)
> > - Replace 'immediate_operand' by 'const_int_operand' in many patterns
> > - Small cleanups in XTheadBb
> > - Factor out C code into thead.cc (XTheadMemPair) to minimize changes in
> > riscv.cc
> >
> > Changes in v3:
> > - Bugfix in XTheadBa
> > - Rewrite of XTheadMemPair
> > - Inclusion of XTheadMemIdx and XTheadFMemIdx
> >
> > Christoph Müllner (9):
> > riscv: Add basic XThead* vendor extension support
> > riscv: riscv-cores.def: Add T-Head XuanTie C906
> > riscv: thead: Add support for the XTheadBa ISA extension
> > riscv: thead: Add support for the XTheadBs ISA extension
> > riscv: thead: Add support for the XTheadBb ISA extension
> > riscv: thead: Add support for the XTheadCondMov ISA extensions
> > riscv: thead: Add support for the XTheadMac ISA extension
> > riscv: thead: Add support for the XTheadFmv ISA extension
> > riscv: thead: Add support for the XTheadMemPair ISA extension
> >
> > gcc/common/config/riscv/riscv-common.cc | 26 ++
> > gcc/config.gcc | 1 +
> > gcc/config/riscv/bitmanip.md | 52 ++-
> > gcc/config/riscv/constraints.md | 8 +
> > gcc/config/riscv/iterators.md | 4 +
> > gcc/config/riscv/peephole.md | 56 +++
> > gcc/config/riscv/riscv-cores.def | 4 +
> > gcc/config/riscv/riscv-opts.h | 26 ++
> > gcc/config/riscv/riscv-protos.h | 16 +-
> > gcc/config/riscv/riscv.cc | 226 +++++++--
> > gcc/config/riscv/riscv.md | 67 ++-
> > gcc/config/riscv/riscv.opt | 3 +
> > gcc/config/riscv/t-riscv | 4 +
> > gcc/config/riscv/thead.cc | 427 ++++++++++++++++++
> > gcc/config/riscv/thead.md | 346 ++++++++++++++
> > .../gcc.target/riscv/mcpu-thead-c906.c | 28 ++
> > .../gcc.target/riscv/xtheadba-addsl.c | 55 +++
> > gcc/testsuite/gcc.target/riscv/xtheadba.c | 14 +
> > gcc/testsuite/gcc.target/riscv/xtheadbb-ext.c | 20 +
> > .../gcc.target/riscv/xtheadbb-extu-2.c | 22 +
> > .../gcc.target/riscv/xtheadbb-extu.c | 22 +
> > gcc/testsuite/gcc.target/riscv/xtheadbb-ff1.c | 18 +
> > gcc/testsuite/gcc.target/riscv/xtheadbb-rev.c | 45 ++
> > .../gcc.target/riscv/xtheadbb-srri.c | 25 +
> > gcc/testsuite/gcc.target/riscv/xtheadbb.c | 14 +
> > gcc/testsuite/gcc.target/riscv/xtheadbs-tst.c | 13 +
> > gcc/testsuite/gcc.target/riscv/xtheadbs.c | 14 +
> > gcc/testsuite/gcc.target/riscv/xtheadcmo.c | 14 +
> > .../riscv/xtheadcondmov-mveqz-imm-eqz.c | 38 ++
> > .../riscv/xtheadcondmov-mveqz-imm-not.c | 38 ++
> > .../riscv/xtheadcondmov-mveqz-reg-eqz.c | 38 ++
> > .../riscv/xtheadcondmov-mveqz-reg-not.c | 38 ++
> > .../riscv/xtheadcondmov-mvnez-imm-cond.c | 38 ++
> > .../riscv/xtheadcondmov-mvnez-imm-nez.c | 38 ++
> > .../riscv/xtheadcondmov-mvnez-reg-cond.c | 38 ++
> > .../riscv/xtheadcondmov-mvnez-reg-nez.c | 38 ++
> > .../gcc.target/riscv/xtheadcondmov.c | 14 +
> > .../gcc.target/riscv/xtheadfmemidx.c | 14 +
> > .../gcc.target/riscv/xtheadfmv-fmv.c | 24 +
> > gcc/testsuite/gcc.target/riscv/xtheadfmv.c | 14 +
> > gcc/testsuite/gcc.target/riscv/xtheadint.c | 14 +
> > .../gcc.target/riscv/xtheadmac-mula-muls.c | 43 ++
> > gcc/testsuite/gcc.target/riscv/xtheadmac.c | 14 +
> > gcc/testsuite/gcc.target/riscv/xtheadmemidx.c | 14 +
> > .../gcc.target/riscv/xtheadmempair-1.c | 98 ++++
> > .../gcc.target/riscv/xtheadmempair-2.c | 84 ++++
> > .../gcc.target/riscv/xtheadmempair-3.c | 29 ++
> > .../gcc.target/riscv/xtheadmempair.c | 13 +
> > gcc/testsuite/gcc.target/riscv/xtheadsync.c | 14 +
> > 49 files changed, 2196 insertions(+), 67 deletions(-)
> > create mode 100644 gcc/config/riscv/thead.cc
> > create mode 100644 gcc/config/riscv/thead.md
> > create mode 100644 gcc/testsuite/gcc.target/riscv/mcpu-thead-c906.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadba-addsl.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadba.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb-ext.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb-extu-2.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb-extu.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb-ff1.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb-rev.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb-srri.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbb.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbs-tst.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadbs.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadcmo.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mveqz-imm-eqz.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mveqz-imm-not.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mveqz-reg-eqz.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mveqz-reg-not.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mvnez-imm-cond.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mvnez-imm-nez.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mvnez-reg-cond.c
> > create mode 100644
> gcc/testsuite/gcc.target/riscv/xtheadcondmov-mvnez-reg-nez.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadcondmov.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadfmemidx.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadfmv-fmv.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadfmv.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadint.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmac-mula-muls.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmac.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmemidx.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmempair-1.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmempair-2.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmempair-3.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadmempair.c
> > create mode 100644 gcc/testsuite/gcc.target/riscv/xtheadsync.c
> >
> > --
> > 2.39.2
> >
>
prev parent reply other threads:[~2023-03-15 9:02 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-02 8:35 Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 1/9] riscv: Add basic XThead* vendor " Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 2/9] riscv: riscv-cores.def: Add T-Head XuanTie C906 Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 3/9] riscv: thead: Add support for the XTheadBa ISA extension Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 4/9] riscv: thead: Add support for the XTheadBs " Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 5/9] riscv: thead: Add support for the XTheadBb " Christoph Muellner
2023-03-05 17:41 ` Jeff Law
2023-03-02 8:35 ` [PATCH v4 6/9] riscv: thead: Add support for the XTheadCondMov ISA extensions Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 7/9] riscv: thead: Add support for the XTheadMac ISA extension Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 8/9] riscv: thead: Add support for the XTheadFmv " Christoph Muellner
2023-03-02 8:35 ` [PATCH v4 9/9] riscv: thead: Add support for the XTheadMemPair " Christoph Muellner
2023-03-05 10:18 ` [PATCH v4 0/9] RISC-V: Add XThead* extension support Kito Cheng
2023-03-15 9:02 ` Philipp Tomsich [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='CAAeLtUDyqj6gXmu=0Qp2W2jMaokCGs_Ff3kCX5S-Jj6aQEyR9A@mail.gmail.com' \
--to=philipp.tomsich@vrull.eu \
--cc=andrew@sifive.com \
--cc=christoph.muellner@vrull.eu \
--cc=cooper.qu@linux.alibaba.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=hp@bitrange.com \
--cc=jim.wilson.gcc@gmail.com \
--cc=kito.cheng@gmail.com \
--cc=kito.cheng@sifive.com \
--cc=lifang_xia@linux.alibaba.com \
--cc=palmer@dabbelt.com \
--cc=pinskia@gmail.com \
--cc=yunhai@linux.alibaba.com \
--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 read-only IMAP folder(s) and NNTP newsgroup(s).