[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-08-06 Thread uros at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

--- Comment #9 from uros at gcc dot gnu.org 2011-08-06 21:37:25 UTC ---
Author: uros
Date: Sat Aug  6 21:37:22 2011
New Revision: 177537

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=177537
Log:
PR testsuite/48727
* g++.dg/opt/devirt2.C: Use specific pattern for alpha*-*-*.


Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/opt/devirt2.C


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-07-14 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

Eric Botcazou  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #8 from Eric Botcazou  2011-07-14 
08:48:19 UTC ---
Everywhere.


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-07-08 Thread ro at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

--- Comment #7 from Rainer Orth  2011-07-08 10:20:39 UTC 
---
Author: ro
Date: Fri Jul  8 10:20:36 2011
New Revision: 176034

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=176034
Log:
PR testsuite/48727
* g++.dg/opt/devirt2.C: Use specific pattern for sparc*-*-*.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/opt/devirt2.C


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-07-07 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

Eric Botcazou  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2011.07.07 13:32:09
 Ever Confirmed|0   |1

--- Comment #6 from Eric Botcazou  2011-07-07 
13:32:09 UTC ---
> Then it's probably best to only look for
> 
> (jmp|call)[^\n]*xyzzy
> 
> on SPARC, and ignore the sethi, so the count still matches?

That seems sufficient, IIUC the point is to have a symbol in the jmp/call.  As
I usually don't test with Sun as, please go ahead with whatever works for you.


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-07-07 Thread ro at CeBiTec dot Uni-Bielefeld.DE
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

--- Comment #5 from ro at CeBiTec dot Uni-Bielefeld.DE  2011-07-07 13:04:18 UTC ---
> --- Comment #4 from Eric Botcazou  2011-07-06 
> 19:15:49 UTC ---
>> This also fails on 32-bit Solaris/SPARC with Sun as, which has:
>> 
>> sethi   %hi(_ZN1S5xyzzyEv), %g1
>> jmp %g1 + %lo(_ZN1S5xyzzyEv)
>> sethi   %hi(_ZN1S5xyzzyEv), %g1
>> jmp %g1 + %lo(_ZN1S5xyzzyEv)
>> 
>> On the other hand, both 64-bit with Sun as and 32-bit with gas have
>> 
>> call_ZN1S5xyzzyEv, 0
>> call_ZN1S5xyzzyEv, 0
>
> This is OK, these are the 2 forms of a direct tail call on the SPARC: 
> sethi/jmp
> or call, depending on the capability of the assembler (HAVE_AS_RELAX_OPTION).

Then it's probably best to only look for

(jmp|call)[^\n]*xyzzy

on SPARC, and ignore the sethi, so the count still matches?  Or, perhaps safer:

(sethi[^\n]*xyzzy[^\n]*\n)?[^\n]*(jmp|call)[^\n]*xyzzy

Rainer


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-07-06 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

--- Comment #4 from Eric Botcazou  2011-07-06 
19:15:49 UTC ---
> This also fails on 32-bit Solaris/SPARC with Sun as, which has:
> 
> sethi   %hi(_ZN1S5xyzzyEv), %g1
> jmp %g1 + %lo(_ZN1S5xyzzyEv)
> sethi   %hi(_ZN1S5xyzzyEv), %g1
> jmp %g1 + %lo(_ZN1S5xyzzyEv)
> 
> On the other hand, both 64-bit with Sun as and 32-bit with gas have
> 
> call_ZN1S5xyzzyEv, 0
> call_ZN1S5xyzzyEv, 0

This is OK, these are the 2 forms of a direct tail call on the SPARC: sethi/jmp
or call, depending on the capability of the assembler (HAVE_AS_RELAX_OPTION).


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-07-06 Thread ro at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

Rainer Orth  changed:

   What|Removed |Added

 Target|hppa2.0w-hp-hpux11.11   |hppa2.0w-hp-hpux11.11,
   ||sparc-sun-solaris2*
 CC||ebotcazou at gcc dot
   ||gnu.org, ro at gcc dot
   ||gnu.org
   Host|hppa2.0w-hp-hpux11.11   |hppa2.0w-hp-hpux11.11,
   ||sparc-sun-solaris2*
   Target Milestone|--- |4.7.0
  Build|hppa2.0w-hp-hpux11.11   |hppa2.0w-hp-hpux11.11,
   ||sparc-sun-solaris2*

--- Comment #3 from Rainer Orth  2011-07-06 17:10:22 UTC 
---
This also fails on 32-bit Solaris/SPARC with Sun as, which has:

sethi   %hi(_ZN1S5xyzzyEv), %g1
jmp %g1 + %lo(_ZN1S5xyzzyEv)
sethi   %hi(_ZN1S5xyzzyEv), %g1
jmp %g1 + %lo(_ZN1S5xyzzyEv)

On the other hand, both 64-bit with Sun as and 32-bit with gas have

call_ZN1S5xyzzyEv, 0
call_ZN1S5xyzzyEv, 0


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-06-16 Thread sje at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

--- Comment #2 from Steve Ellcey  2011-06-16 20:38:52 
UTC ---
Author: sje
Date: Thu Jun 16 20:38:49 2011
New Revision: 175112

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=175112
Log:
2011-06-16  Steve Ellcey  

PR testsuite/48727
* g++.dg/opt/devirt2.C: Change xfail rule to target.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/opt/devirt2.C


[Bug testsuite/48727] FAIL: g++.dg/opt/devirt2.C scan-assembler-times xyzzy 2

2011-06-14 Thread sje at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48727

--- Comment #1 from Steve Ellcey  2011-06-14 20:26:13 
UTC ---
Author: sje
Date: Tue Jun 14 20:26:08 2011
New Revision: 175055

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=175055
Log:
2011-06-14  Steve Ellcey  

PR testsuite/48727
* g++.dg/opt/devirt2.C: Fix scan rules for ia64*-*-hpux* and hppa*-*-*.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/opt/devirt2.C