Re: [PATCH 08/24] Make gdb.base/sizeof.exp use gdb_test_stdio

Pedro Alves
2015-04-23T15:49:47Z

On 04/23/2015 04:38 PM, Antoine Tremblay wrote:

>> It's not part of the patch but should we replace :
>>
>> # Test depends on printf, which the sparclet stub doesn't support.
>> if { [istarget "sparclet-*-*"] } {
>> return 0
>> }
>>
>> With :if [target_info exists gdb,noinferiorio] { ?
>>

Yes, though...

>> This is present in other tests too...
>>
>> Thanks,
>> Antoine
>>
>>
>> And add that option to the sparclet target ? ( I could not find that
>> target in the tree even if config/m32r-stub.exp references it...

... I had never heard of sparclet before, until I noticed that line too.
I had assumed it was a Sun SPARC cpu variant, and ignored it.

But now I notice that it's something else. We still have a chapter
about it in the manual:

https://sourceware.org/gdb/onlinedocs/gdb/Sparclet.html

But, NEWS says the support was removed in gdb 6.1...

* REMOVED configurations and files

...
Tsqware Sparclet sparclet-*-*
...

ChangeLog-2003 has:

...
* sparclet-stub.c, sparclet-rom.c: Delete obsolete file.
...

etc.

So the remaining bits are just waiting for someone to garbage
collect them.

>>
>
> Oops I've hit send by mistake this was meant for [PATCH 09/24] Make
> gdb.base/call-strs.exp use gdb_test_stdio and others like it...
>
> Sorry about that
>

No worries. :-)

Thanks,
Pedro Alves