Re: Testsuite on Solaris 11.3/sparc

[Gaius Mulley]

john o goyo <jog37@riddermarkfarm.ca> writes:

> Greetings, Gaius.
>
> On 2023-04-22 08:36, john o goyo wrote:
>> Greetings, Gaius.
>>
>> On 2023-04-21 09:34, Gaius Mulley wrote:
>>> [...]
>>> Hi John,
>>>
>>> re: (1) here is a patch which might be along the right lines?
>>>
>>> diff --git a/gcc/testsuite/lib/gm2.exp b/gcc/testsuite/lib/gm2.exp
>>> index 25b5e684b3a..9ed8c6ebd53 100644
>>> --- a/gcc/testsuite/lib/gm2.exp
>>> +++ b/gcc/testsuite/lib/gm2.exp
>>> @@ -218,6 +218,10 @@ proc gm2_target_compile_default { source dest
>>> type options } {
>>>       if [info exists TOOL_OPTIONS] {
>>>       lappend options "additional_flags=$TOOL_OPTIONS"
>>>       }
>>> +    if [istarget *-*-solaris2*] {
>>> +    lappend options "ldflags=-lnsl -lsocket"
>>> +    }
>>> +
>>>       lappend options "timeout=[timeout_value]"
>>>       lappend options "compiler=$GCC_UNDER_TEST"
>>>       # puts stderr "options = $options\n"
>>>
>>>
>>> I'm not sure if the solaris2 string is correct though.  But it maybe
>>> worth just forcing lappend options "ldflags=-lnsl -lsocket" as an
>>> experiment and refining afterwards.
>>>
>>> Hope this helps?
>>>
>>> regards,
>>> Gaius
>> Thank you. I added the options unreserved as you suggested and this
>> elimated 8 failures:
>>
>>                 === gm2 Summary ===
>>
>> # of expected passes            11874
>> # of unexpected failures        19
>> # of unresolved testcases       7
>>
>> (Note that I am building gcc-13.1.0-RC2-20230419.)
>>
>> Sincerely,
>> john
>>
>>
> The string about is correct.  The same results from adding the conditional.
>

Hi John,

thanks for testing - I'll apply the patch!

regards,
Gaius