[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

Jakub Jelinek  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 CC||jakub at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #6 from Jakub Jelinek  ---
Fixed.

[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

--- Comment #5 from Martin Liška  ---
Author: marxin
Date: Wed Feb 28 13:45:59 2018
New Revision: 258063

URL: https://gcc.gnu.org/viewcvs?rev=258063=gcc=rev
Log:
Fix timevar format and fix tests (PR testsuite/84597).

2018-02-28  Martin Liska  

PR testsuite/84597
* timevar.c (timer::print): Fix format to properly print 100%
values.
2018-02-28  Martin Liska  

PR testsuite/84597
* g++.dg/ext/timevar1.C: Fix pruned patterns.
* g++.dg/ext/timevar2.C: Likewise.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/ext/timevar1.C
trunk/gcc/testsuite/g++.dg/ext/timevar2.C
trunk/gcc/timevar.c

[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|--- |8.0

[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[clyon at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

Christophe Lyon  changed:

   What|Removed |Added

 CC||clyon at gcc dot gnu.org

--- Comment #4 from Christophe Lyon  ---
(In reply to Martin Liška from comment #3)
> Sorry for the breakage, let me fix it.

I think it's just a matter of updating the testcases and replace 'times' with
'variable' since your patch replaced 'Execution times (seconds)' with 'Time
variable'.

[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2018-02-28
   Assignee|unassigned at gcc dot gnu.org  |marxin at gcc dot 
gnu.org
 Ever confirmed|0   |1

--- Comment #3 from Martin Liška  ---
Sorry for the breakage, let me fix it.

[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[nathan at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

Nathan Sidwell  changed:

   What|Removed |Added

 CC||nathan at gcc dot gnu.org

--- Comment #2 from Nathan Sidwell  ---
Seeing this on x86_64-linux too FWIW

[Bug testsuite/84597] [8 regression] test case g++.dg/ext/timevar2.C fails starting with r258029

[seurer at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84597

--- Comment #1 from seurer at gcc dot gnu.org ---
I missed that timevar1.C is also failing.

> FAIL: g++.dg/ext/timevar1.C  -std=gnu++11 (test for excess errors)
> FAIL: g++.dg/ext/timevar1.C  -std=gnu++14 (test for excess errors)
> FAIL: g++.dg/ext/timevar1.C  -std=gnu++98 (test for excess errors)
> FAIL: g++.dg/ext/timevar2.C  -std=gnu++11 (test for excess errors)
> FAIL: g++.dg/ext/timevar2.C  -std=gnu++14 (test for excess errors)
> FAIL: g++.dg/ext/timevar2.C  -std=gnu++98 (test for excess errors)