* [PATCH] RISC-V: Document optimization parameter riscv-strcmp-inline-limit
@ 2023-12-02 21:15 Christoph Müllner
2023-12-04 2:43 ` Kito Cheng
0 siblings, 1 reply; 4+ messages in thread
From: Christoph Müllner @ 2023-12-02 21:15 UTC (permalink / raw)
To: gcc-patches, Kito Cheng, Jim Wilson, Palmer Dabbelt,
Andrew Waterman, Philipp Tomsich, Jeff Law, Jeremy Bennett
Cc: Christoph Müllner
This patch documents the optimization parameter
riscv-strcmp-inline-limit, which can be used to tweak the behaviour
of -minline-strcmp and -minline-strncmp.
gcc/ChangeLog:
PR target/112650
* doc/invoke.texi: Document riscv-strcmp-inline-limit.
Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
---
gcc/doc/invoke.texi | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 2fab4c5d71f..ba2d843b484 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -29846,6 +29846,10 @@ Inlining will only be done if the strings are properly aligned
and instructions for accelerated processing are available.
The default is to not inline strcmp calls.
+The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
+the maximum number of bytes compared by the inlined code.
+The default value is 64.
+
@opindex minline-strncmp
@item -minline-strncmp
@itemx -mno-inline-strncmp
@@ -29854,6 +29858,10 @@ Inlining will only be done if the strings are properly aligned
and instructions for accelerated processing are available.
The default is to not inline strncmp calls.
+The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
+the maximum number of bytes compared by the inlined code.
+The default value is 64.
+
@opindex mshorten-memrefs
@item -mshorten-memrefs
@itemx -mno-shorten-memrefs
--
2.41.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] RISC-V: Document optimization parameter riscv-strcmp-inline-limit
2023-12-02 21:15 [PATCH] RISC-V: Document optimization parameter riscv-strcmp-inline-limit Christoph Müllner
@ 2023-12-04 2:43 ` Kito Cheng
2023-12-04 3:46 ` Kito Cheng
0 siblings, 1 reply; 4+ messages in thread
From: Kito Cheng @ 2023-12-04 2:43 UTC (permalink / raw)
To: Christoph Müllner
Cc: gcc-patches, Kito Cheng, Jim Wilson, Palmer Dabbelt,
Andrew Waterman, Philipp Tomsich, Jeff Law, Jeremy Bennett
[-- Attachment #1: Type: text/plain, Size: 1652 bytes --]
LGTM
On Sun, Dec 3, 2023 at 5:16 AM Christoph Müllner <
christoph.muellner@vrull.eu> wrote:
> This patch documents the optimization parameter
> riscv-strcmp-inline-limit, which can be used to tweak the behaviour
> of -minline-strcmp and -minline-strncmp.
>
> gcc/ChangeLog:
>
> PR target/112650
> * doc/invoke.texi: Document riscv-strcmp-inline-limit.
>
> Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
> ---
> gcc/doc/invoke.texi | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> index 2fab4c5d71f..ba2d843b484 100644
> --- a/gcc/doc/invoke.texi
> +++ b/gcc/doc/invoke.texi
> @@ -29846,6 +29846,10 @@ Inlining will only be done if the strings are
> properly aligned
> and instructions for accelerated processing are available.
> The default is to not inline strcmp calls.
>
> +The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
> +the maximum number of bytes compared by the inlined code.
> +The default value is 64.
> +
> @opindex minline-strncmp
> @item -minline-strncmp
> @itemx -mno-inline-strncmp
> @@ -29854,6 +29858,10 @@ Inlining will only be done if the strings are
> properly aligned
> and instructions for accelerated processing are available.
> The default is to not inline strncmp calls.
>
> +The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
> +the maximum number of bytes compared by the inlined code.
> +The default value is 64.
> +
> @opindex mshorten-memrefs
> @item -mshorten-memrefs
> @itemx -mno-shorten-memrefs
> --
> 2.41.0
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] RISC-V: Document optimization parameter riscv-strcmp-inline-limit
2023-12-04 2:43 ` Kito Cheng
@ 2023-12-04 3:46 ` Kito Cheng
2023-12-04 9:56 ` Christoph Müllner
0 siblings, 1 reply; 4+ messages in thread
From: Kito Cheng @ 2023-12-04 3:46 UTC (permalink / raw)
To: Christoph Müllner
Cc: gcc-patches, Kito Cheng, Jim Wilson, Palmer Dabbelt,
Andrew Waterman, Philipp Tomsich, Jeff Law, Jeremy Bennett
Wait, I got this on my machine?
../../../../riscv-gnu-toolchain-trunk/gcc/gcc/doc/invoke.texi:29774: misplaced }
../../../../riscv-gnu-toolchain-trunk/gcc/gcc/doc/invoke.texi:29786: misplaced }
On Mon, Dec 4, 2023 at 10:43 AM Kito Cheng <kito.cheng@gmail.com> wrote:
>
> LGTM
>
> On Sun, Dec 3, 2023 at 5:16 AM Christoph Müllner <christoph.muellner@vrull.eu> wrote:
>>
>> This patch documents the optimization parameter
>> riscv-strcmp-inline-limit, which can be used to tweak the behaviour
>> of -minline-strcmp and -minline-strncmp.
>>
>> gcc/ChangeLog:
>>
>> PR target/112650
>> * doc/invoke.texi: Document riscv-strcmp-inline-limit.
>>
>> Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
>> ---
>> gcc/doc/invoke.texi | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>>
>> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
>> index 2fab4c5d71f..ba2d843b484 100644
>> --- a/gcc/doc/invoke.texi
>> +++ b/gcc/doc/invoke.texi
>> @@ -29846,6 +29846,10 @@ Inlining will only be done if the strings are properly aligned
>> and instructions for accelerated processing are available.
>> The default is to not inline strcmp calls.
>>
>> +The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
>> +the maximum number of bytes compared by the inlined code.
>> +The default value is 64.
>> +
>> @opindex minline-strncmp
>> @item -minline-strncmp
>> @itemx -mno-inline-strncmp
>> @@ -29854,6 +29858,10 @@ Inlining will only be done if the strings are properly aligned
>> and instructions for accelerated processing are available.
>> The default is to not inline strncmp calls.
>>
>> +The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
>> +the maximum number of bytes compared by the inlined code.
>> +The default value is 64.
>> +
>> @opindex mshorten-memrefs
>> @item -mshorten-memrefs
>> @itemx -mno-shorten-memrefs
>> --
>> 2.41.0
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] RISC-V: Document optimization parameter riscv-strcmp-inline-limit
2023-12-04 3:46 ` Kito Cheng
@ 2023-12-04 9:56 ` Christoph Müllner
0 siblings, 0 replies; 4+ messages in thread
From: Christoph Müllner @ 2023-12-04 9:56 UTC (permalink / raw)
To: Kito Cheng
Cc: gcc-patches, Kito Cheng, Jim Wilson, Palmer Dabbelt,
Andrew Waterman, Philipp Tomsich, Jeff Law, Jeremy Bennett
On Mon, Dec 4, 2023 at 4:46 AM Kito Cheng <kito.cheng@gmail.com> wrote:
>
> Wait, I got this on my machine?
>
> ../../../../riscv-gnu-toolchain-trunk/gcc/gcc/doc/invoke.texi:29774: misplaced }
> ../../../../riscv-gnu-toolchain-trunk/gcc/gcc/doc/invoke.texi:29786: misplaced }
@{n} should be @var{n}.
I was too optimistic and sent the patch before the build finished (or
in this case failed).
Sorry for that.
I have sent a v2 that builds fine:
https://gcc.gnu.org/pipermail/gcc-patches/2023-December/639142.html
>
>
> On Mon, Dec 4, 2023 at 10:43 AM Kito Cheng <kito.cheng@gmail.com> wrote:
> >
> > LGTM
> >
> > On Sun, Dec 3, 2023 at 5:16 AM Christoph Müllner <christoph.muellner@vrull.eu> wrote:
> >>
> >> This patch documents the optimization parameter
> >> riscv-strcmp-inline-limit, which can be used to tweak the behaviour
> >> of -minline-strcmp and -minline-strncmp.
> >>
> >> gcc/ChangeLog:
> >>
> >> PR target/112650
> >> * doc/invoke.texi: Document riscv-strcmp-inline-limit.
> >>
> >> Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
> >> ---
> >> gcc/doc/invoke.texi | 8 ++++++++
> >> 1 file changed, 8 insertions(+)
> >>
> >> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> >> index 2fab4c5d71f..ba2d843b484 100644
> >> --- a/gcc/doc/invoke.texi
> >> +++ b/gcc/doc/invoke.texi
> >> @@ -29846,6 +29846,10 @@ Inlining will only be done if the strings are properly aligned
> >> and instructions for accelerated processing are available.
> >> The default is to not inline strcmp calls.
> >>
> >> +The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
> >> +the maximum number of bytes compared by the inlined code.
> >> +The default value is 64.
> >> +
> >> @opindex minline-strncmp
> >> @item -minline-strncmp
> >> @itemx -mno-inline-strncmp
> >> @@ -29854,6 +29858,10 @@ Inlining will only be done if the strings are properly aligned
> >> and instructions for accelerated processing are available.
> >> The default is to not inline strncmp calls.
> >>
> >> +The @option{--param riscv-strcmp-inline-limit=@{n}} parameter controls
> >> +the maximum number of bytes compared by the inlined code.
> >> +The default value is 64.
> >> +
> >> @opindex mshorten-memrefs
> >> @item -mshorten-memrefs
> >> @itemx -mno-shorten-memrefs
> >> --
> >> 2.41.0
> >>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-12-04 9:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-02 21:15 [PATCH] RISC-V: Document optimization parameter riscv-strcmp-inline-limit Christoph Müllner
2023-12-04 2:43 ` Kito Cheng
2023-12-04 3:46 ` Kito Cheng
2023-12-04 9:56 ` Christoph Müllner
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).