* [PATCH FYI] -finline-stringops: copy timeout factor from memcmp-1.c test
[not found] <3d8665b0-d9c0-4da6-bb92-194e2b68494c@ventanamicro.com>
@ 2023-12-20 2:32 ` Alexandre Oliva
2023-12-20 3:51 ` Jeff Law
0 siblings, 1 reply; 2+ messages in thread
From: Alexandre Oliva @ 2023-12-20 2:32 UTC (permalink / raw)
To: Jeff Law; +Cc: gcc-patches
Hi, Jeff,
On Dec 18, 2023, Jeff Law <jlaw@ventanamicro.com> wrote:
> These are timing sporadically on the embedded platforms. Given they
> include a test that has a timeout factor, it seems to me you should
> duplicate the timeout factor in the new tests.
> Remember when you include another file, the dg- directives in the
> other file aren't applied.
Thanks for the reminder. Sorry I missed most of them.
I added some -finline-stringops tests that included memcmp-1.c, but
carried over the timeout factor onto only one such test. Jeff Law
kindly pointed that out (thanks!), so here's the fix.
Testing on x86_64-linux-gnu. I'll check this in as obvious once testing
is done.
for gcc/testsuite/ChangeLog
* gcc.dg/torture/inline-mem-cmp-1.c: Copy timeout factor from
mem-cmp-1.c.
* gcc.dg/torture/inline-mem-cpy-1.c: Likewise.
---
gcc/testsuite/gcc.dg/torture/inline-mem-cmp-1.c | 1 +
gcc/testsuite/gcc.dg/torture/inline-mem-cpy-1.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/gcc/testsuite/gcc.dg/torture/inline-mem-cmp-1.c b/gcc/testsuite/gcc.dg/torture/inline-mem-cmp-1.c
index a368f0741129d..4bc66597b35a6 100644
--- a/gcc/testsuite/gcc.dg/torture/inline-mem-cmp-1.c
+++ b/gcc/testsuite/gcc.dg/torture/inline-mem-cmp-1.c
@@ -1,5 +1,6 @@
/* { dg-do run } */
/* { dg-options "-finline-stringops=memcmp -save-temps -g0 -fno-lto" } */
+/* { dg-timeout-factor 2 } */
#include "../memcmp-1.c"
diff --git a/gcc/testsuite/gcc.dg/torture/inline-mem-cpy-1.c b/gcc/testsuite/gcc.dg/torture/inline-mem-cpy-1.c
index c98e903c1f169..f4952554dd011 100644
--- a/gcc/testsuite/gcc.dg/torture/inline-mem-cpy-1.c
+++ b/gcc/testsuite/gcc.dg/torture/inline-mem-cpy-1.c
@@ -1,5 +1,6 @@
/* { dg-do run } */
/* { dg-options "-finline-stringops=memcpy -save-temps -g0 -fno-lto" } */
+/* { dg-timeout-factor 2 } */
#include "../memcmp-1.c"
/* Yeah, this memcmp test exercises plenty of memcpy, more than any of the
--
Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/
Free Software Activist GNU Toolchain Engineer
Disinformation flourishes because many people care deeply about injustice but
very few check the facts. Think Assange & Stallman. The empires strike back
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH FYI] -finline-stringops: copy timeout factor from memcmp-1.c test
2023-12-20 2:32 ` [PATCH FYI] -finline-stringops: copy timeout factor from memcmp-1.c test Alexandre Oliva
@ 2023-12-20 3:51 ` Jeff Law
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Law @ 2023-12-20 3:51 UTC (permalink / raw)
To: Alexandre Oliva, Jeff Law; +Cc: gcc-patches
On 12/19/23 19:32, Alexandre Oliva wrote:
> Hi, Jeff,
>
> On Dec 18, 2023, Jeff Law <jlaw@ventanamicro.com> wrote:
>
>> These are timing sporadically on the embedded platforms. Given they
>> include a test that has a timeout factor, it seems to me you should
>> duplicate the timeout factor in the new tests.
>
>> Remember when you include another file, the dg- directives in the
>> other file aren't applied.
>
> Thanks for the reminder. Sorry I missed most of them.
>
>
> I added some -finline-stringops tests that included memcmp-1.c, but
> carried over the timeout factor onto only one such test. Jeff Law
> kindly pointed that out (thanks!), so here's the fix.
>
> Testing on x86_64-linux-gnu. I'll check this in as obvious once testing
> is done.
>
>
> for gcc/testsuite/ChangeLog
>
> * gcc.dg/torture/inline-mem-cmp-1.c: Copy timeout factor from
> mem-cmp-1.c.
> * gcc.dg/torture/inline-mem-cpy-1.c: Likewise.
Thanks.
jeff
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-12-20 3:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <3d8665b0-d9c0-4da6-bb92-194e2b68494c@ventanamicro.com>
2023-12-20 2:32 ` [PATCH FYI] -finline-stringops: copy timeout factor from memcmp-1.c test Alexandre Oliva
2023-12-20 3:51 ` Jeff Law
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).