* Re: [PATCH] string: Add a testcase for wcsncmp with SIZE_MAX [BZ #28755] [not found] ` <CAMAf5_ei=o4i2GKe07xSQg+32uA5Bq+PB3TNNwchTkr0zHevMg@mail.gmail.com> @ 2022-02-17 19:05 ` H.J. Lu 2022-02-17 19:17 ` Noah Goldstein 0 siblings, 1 reply; 3+ messages in thread From: H.J. Lu @ 2022-02-17 19:05 UTC (permalink / raw) To: Sunil Pandey, Libc-stable Mailing List; +Cc: GNU C Library On Thu, Feb 17, 2022 at 10:58 AM Sunil Pandey <skpgkp2@gmail.com> wrote: > > LGTM > > Reviewed-by: Sunil K Pandey <skpgkp2@gmail.com> I am backporting this to release branches. > On Thu, Feb 17, 2022 at 8:28 AM H.J. Lu via Libc-alpha > <libc-alpha@sourceware.org> wrote: > > > > Verify that wcsncmp (L"abc", L"abd", SIZE_MAX) == 0. The new test fails > > without > > > > commit ddf0992cf57a93200e0c782e2a94d0733a5a0b87 > > Author: Noah Goldstein <goldstein.w.n@gmail.com> > > Date: Sun Jan 9 16:02:21 2022 -0600 > > > > x86: Fix __wcsncmp_avx2 in strcmp-avx2.S [BZ# 28755] > > > > and > > > > commit 7e08db3359c86c94918feb33a1182cd0ff3bb10b > > Author: Noah Goldstein <goldstein.w.n@gmail.com> > > Date: Sun Jan 9 16:02:28 2022 -0600 > > > > x86: Fix __wcsncmp_evex in strcmp-evex.S [BZ# 28755] > > > > This is for BZ #28755. > > --- > > string/test-strncmp.c | 13 +++++++++++++ > > 1 file changed, 13 insertions(+) > > > > diff --git a/string/test-strncmp.c b/string/test-strncmp.c > > index df7cea4068..1a87f0e73e 100644 > > --- a/string/test-strncmp.c > > +++ b/string/test-strncmp.c > > @@ -445,6 +445,18 @@ check4 (void) > > check_result (impl, s1, s2, size, exp_result); > > } > > > > +static void > > +check5 (void) > > +{ > > + const CHAR *s1 = L ("abc"); > > + CHAR *s2 = STRDUP (s1); > > + > > + FOR_EACH_IMPL (impl, 0) > > + check_result (impl, s1, s2, SIZE_MAX, 0); > > + > > + free (s2); > > +} > > + > > static void > > check_overflow (void) > > { > > @@ -569,6 +581,7 @@ test_main (void) > > check2 (); > > check3 (); > > check4 (); > > + check5 (); > > > > printf ("%23s", ""); > > FOR_EACH_IMPL (impl, 0) > > -- > > 2.35.1 > > -- H.J. ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] string: Add a testcase for wcsncmp with SIZE_MAX [BZ #28755] 2022-02-17 19:05 ` [PATCH] string: Add a testcase for wcsncmp with SIZE_MAX [BZ #28755] H.J. Lu @ 2022-02-17 19:17 ` Noah Goldstein 2022-02-17 19:22 ` H.J. Lu 0 siblings, 1 reply; 3+ messages in thread From: Noah Goldstein @ 2022-02-17 19:17 UTC (permalink / raw) To: H.J. Lu; +Cc: Sunil Pandey, Libc-stable Mailing List, GNU C Library On Thu, Feb 17, 2022 at 1:06 PM H.J. Lu via Libc-alpha <libc-alpha@sourceware.org> wrote: > > On Thu, Feb 17, 2022 at 10:58 AM Sunil Pandey <skpgkp2@gmail.com> wrote: > > > > LGTM > > > > Reviewed-by: Sunil K Pandey <skpgkp2@gmail.com> > > I am backporting this to release branches. We will also need to backport the change of target for the RTM + AVX2 versions. > > > On Thu, Feb 17, 2022 at 8:28 AM H.J. Lu via Libc-alpha > > <libc-alpha@sourceware.org> wrote: > > > > > > Verify that wcsncmp (L"abc", L"abd", SIZE_MAX) == 0. The new test fails > > > without > > > > > > commit ddf0992cf57a93200e0c782e2a94d0733a5a0b87 > > > Author: Noah Goldstein <goldstein.w.n@gmail.com> > > > Date: Sun Jan 9 16:02:21 2022 -0600 > > > > > > x86: Fix __wcsncmp_avx2 in strcmp-avx2.S [BZ# 28755] > > > > > > and > > > > > > commit 7e08db3359c86c94918feb33a1182cd0ff3bb10b > > > Author: Noah Goldstein <goldstein.w.n@gmail.com> > > > Date: Sun Jan 9 16:02:28 2022 -0600 > > > > > > x86: Fix __wcsncmp_evex in strcmp-evex.S [BZ# 28755] > > > > > > This is for BZ #28755. > > > --- > > > string/test-strncmp.c | 13 +++++++++++++ > > > 1 file changed, 13 insertions(+) > > > > > > diff --git a/string/test-strncmp.c b/string/test-strncmp.c > > > index df7cea4068..1a87f0e73e 100644 > > > --- a/string/test-strncmp.c > > > +++ b/string/test-strncmp.c > > > @@ -445,6 +445,18 @@ check4 (void) > > > check_result (impl, s1, s2, size, exp_result); > > > } > > > > > > +static void > > > +check5 (void) > > > +{ > > > + const CHAR *s1 = L ("abc"); > > > + CHAR *s2 = STRDUP (s1); > > > + > > > + FOR_EACH_IMPL (impl, 0) > > > + check_result (impl, s1, s2, SIZE_MAX, 0); > > > + > > > + free (s2); > > > +} > > > + > > > static void > > > check_overflow (void) > > > { > > > @@ -569,6 +581,7 @@ test_main (void) > > > check2 (); > > > check3 (); > > > check4 (); > > > + check5 (); > > > > > > printf ("%23s", ""); > > > FOR_EACH_IMPL (impl, 0) > > > -- > > > 2.35.1 > > > > > > > -- > H.J. ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] string: Add a testcase for wcsncmp with SIZE_MAX [BZ #28755] 2022-02-17 19:17 ` Noah Goldstein @ 2022-02-17 19:22 ` H.J. Lu 0 siblings, 0 replies; 3+ messages in thread From: H.J. Lu @ 2022-02-17 19:22 UTC (permalink / raw) To: Noah Goldstein; +Cc: Sunil Pandey, Libc-stable Mailing List, GNU C Library On Thu, Feb 17, 2022 at 11:17 AM Noah Goldstein <goldstein.w.n@gmail.com> wrote: > > On Thu, Feb 17, 2022 at 1:06 PM H.J. Lu via Libc-alpha > <libc-alpha@sourceware.org> wrote: > > > > On Thu, Feb 17, 2022 at 10:58 AM Sunil Pandey <skpgkp2@gmail.com> wrote: > > > > > > LGTM > > > > > > Reviewed-by: Sunil K Pandey <skpgkp2@gmail.com> > > > > I am backporting this to release branches. > We will also need to backport the change of target for the RTM + AVX2 versions. Can you do that after it is landed on master branch? Thanks. > > > > > > On Thu, Feb 17, 2022 at 8:28 AM H.J. Lu via Libc-alpha > > > <libc-alpha@sourceware.org> wrote: > > > > > > > > Verify that wcsncmp (L"abc", L"abd", SIZE_MAX) == 0. The new test fails > > > > without > > > > > > > > commit ddf0992cf57a93200e0c782e2a94d0733a5a0b87 > > > > Author: Noah Goldstein <goldstein.w.n@gmail.com> > > > > Date: Sun Jan 9 16:02:21 2022 -0600 > > > > > > > > x86: Fix __wcsncmp_avx2 in strcmp-avx2.S [BZ# 28755] > > > > > > > > and > > > > > > > > commit 7e08db3359c86c94918feb33a1182cd0ff3bb10b > > > > Author: Noah Goldstein <goldstein.w.n@gmail.com> > > > > Date: Sun Jan 9 16:02:28 2022 -0600 > > > > > > > > x86: Fix __wcsncmp_evex in strcmp-evex.S [BZ# 28755] > > > > > > > > This is for BZ #28755. > > > > --- > > > > string/test-strncmp.c | 13 +++++++++++++ > > > > 1 file changed, 13 insertions(+) > > > > > > > > diff --git a/string/test-strncmp.c b/string/test-strncmp.c > > > > index df7cea4068..1a87f0e73e 100644 > > > > --- a/string/test-strncmp.c > > > > +++ b/string/test-strncmp.c > > > > @@ -445,6 +445,18 @@ check4 (void) > > > > check_result (impl, s1, s2, size, exp_result); > > > > } > > > > > > > > +static void > > > > +check5 (void) > > > > +{ > > > > + const CHAR *s1 = L ("abc"); > > > > + CHAR *s2 = STRDUP (s1); > > > > + > > > > + FOR_EACH_IMPL (impl, 0) > > > > + check_result (impl, s1, s2, SIZE_MAX, 0); > > > > + > > > > + free (s2); > > > > +} > > > > + > > > > static void > > > > check_overflow (void) > > > > { > > > > @@ -569,6 +581,7 @@ test_main (void) > > > > check2 (); > > > > check3 (); > > > > check4 (); > > > > + check5 (); > > > > > > > > printf ("%23s", ""); > > > > FOR_EACH_IMPL (impl, 0) > > > > -- > > > > 2.35.1 > > > > > > > > > > > > -- > > H.J. -- H.J. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-02-17 19:22 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20220217162831.417972-1-hjl.tools@gmail.com> [not found] ` <CAMAf5_ei=o4i2GKe07xSQg+32uA5Bq+PB3TNNwchTkr0zHevMg@mail.gmail.com> 2022-02-17 19:05 ` [PATCH] string: Add a testcase for wcsncmp with SIZE_MAX [BZ #28755] H.J. Lu 2022-02-17 19:17 ` Noah Goldstein 2022-02-17 19:22 ` H.J. Lu
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).