Patchwork [testsuite] : Do not run simulate-thread on powerpc*-*-darwin*

login
register
mail settings
Submitter IainS
Date Nov. 13, 2011, 9:32 p.m.
Message ID <06F5B6F4-8EFD-480D-B18E-CD65CB6F46D4@sandoe-acoustics.co.uk>
Download mbox | patch
Permalink /patch/125453/
State New
Headers show

Comments

IainS - Nov. 13, 2011, 9:32 p.m.
It seems like:

http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01543.html

we have a problem on powerpc-darwin9
(although i686-darwin9 and x86-64-darwin10 seem OK)

this prevents the test-suite finishing with gdb just spinning... and  
you have to kill each invocation by hand (seems like the expect  
timeout logic is not working somehow)....

for now, it would be good to disable this - to allow progress

(I have a partial patch to gdb 7.31 that allows it to work with  
powerpc-darwin9 -- but not ready yet).

OK for trunk?
Iain

gcc/testsuite:

	* lib/gcc-simulate-thread.exp (simulate-thread): Don't run for
	powerpc*-*darwin*.

      if { ![isnative] || [is_remote target] } { return }
Mike Stump - Nov. 14, 2011, 6:39 p.m.
On Nov 13, 2011, at 1:32 PM, Iain Sandoe <developer@sandoe-acoustics.co.uk> wrote:
> we have a problem on powerpc-darwin9

> OK for trunk?

Ok, please. Include a comment line just before staying why.

Patch

Index: gcc/testsuite/lib/gcc-simulate-thread.exp
===================================================================
--- gcc/testsuite/lib/gcc-simulate-thread.exp	(revision 181339)
+++ gcc/testsuite/lib/gcc-simulate-thread.exp	(working copy)
@@ -26,6 +26,7 @@  proc simulate-thread { args } {
      # ??? Exit immediately if this is alpha*-*-linux* target, single- 
stepping
      # executable between ldl_l and stl_c insns in gdb breaks LL/SC  
chaining.
      if { [istarget alpha*-*-linux*] } { return }
+    if { [istarget powerpc*-*-darwin*] } { return }