public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Adhemerval Zanella Netto <adhemerval.zanella@linaro.org>
To: "Christoph Müllner" <christoph.muellner@vrull.eu>
Cc: libc-alpha@sourceware.org, Palmer Dabbelt <palmer@dabbelt.com>,
	Darius Rad <darius@bluespec.com>,
	Andrew Waterman <andrew@sifive.com>,
	Philipp Tomsich <philipp.tomsich@vrull.eu>
Subject: Re: [PATCH] riscv: Add support for XTheadBb in string-fz[a,i].h
Date: Wed, 6 Sep 2023 08:39:51 -0300	[thread overview]
Message-ID: <5bf7d52a-a738-4e14-6d2a-0291edf00169@linaro.org> (raw)
In-Reply-To: <CAEg0e7jm77-_pGSWhhGZF5Js3qK2rVcDY3KsuDecUTRQXVZYQA@mail.gmail.com>



On 06/09/23 07:34, Christoph Müllner wrote:
> 
> 
> On Thu, Aug 24, 2023 at 7:21 PM Adhemerval Zanella Netto <adhemerval.zanella@linaro.org <mailto:adhemerval.zanella@linaro.org>> wrote:
> 
> 
> 
>     On 23/08/23 02:46, Christoph Muellner wrote:
>     > From: Christoph Müllner <christoph.muellner@vrull.eu <mailto:christoph.muellner@vrull.eu>>
>     >
>     > XTheadBb has similar instructions like Zbb, which allow optimized
>     > string processing:
>     > * th.ff0: find-first zero is a CLZ instruction.
>     > * th.tstnbz: Similar like orc.b, but with a bit-inverted result.
>     >
>     > The instructions are documented here:
>     >   https://github.com/T-head-Semi/thead-extension-spec/tree/master/xtheadbb <https://github.com/T-head-Semi/thead-extension-spec/tree/master/xtheadbb>
>     >
>     > These instructions can be found in the T-Head C906 and the C910.
>     >
>     > Tested with the string tests.
>     >
>     > Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu <mailto:christoph.muellner@vrull.eu>>
> 
> 
>     LGTM, thanks.  Is th.tstnbz available as builtin by chance?
> 
>     Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org <mailto:adhemerval.zanella@linaro.org>>
> 
> 

Someone to actually push it, since it is already reviewed. I will install it.

  reply	other threads:[~2023-09-06 11:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-23  5:46 Christoph Muellner
2023-08-24 17:21 ` Adhemerval Zanella Netto
2023-08-24 17:36   ` Christoph Müllner
2023-09-06 10:34   ` Christoph Müllner
2023-09-06 11:39     ` Adhemerval Zanella Netto [this message]
2023-09-06 13:52       ` Christoph Müllner

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=5bf7d52a-a738-4e14-6d2a-0291edf00169@linaro.org \
    --to=adhemerval.zanella@linaro.org \
    --cc=andrew@sifive.com \
    --cc=christoph.muellner@vrull.eu \
    --cc=darius@bluespec.com \
    --cc=libc-alpha@sourceware.org \
    --cc=palmer@dabbelt.com \
    --cc=philipp.tomsich@vrull.eu \
    /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).