Re: [PATCH v2 01/47] gdb, intelgt: add intelgt as a basic machine

[Jan Beulich <jbeulich@suse.com>]

On 17.12.2024 19:48, Aktemur, Tankut Baris wrote:
> On Monday, December 16, 2024 8:54 AM, Jan Beulich wrote:
>> On 13.12.2024 16:59, Tankut Baris Aktemur wrote:
>>> Add 'intelgt' as a basic machine to config.sub.
>>>
>>> To: <config-patches@gnu.org>
>>> To: <binutils@sourceware.org>
>>> ---
>>>  config.sub | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/config.sub b/config.sub
>>> index 2c6a07ab3c34eabed8318ec0a37c0cc23b77a63f..63ff958ec125e543674e9b261d5e5bb2fa749c4e
>> 100755
>>> --- a/config.sub
>>> +++ b/config.sub
>>> @@ -1205,6 +1205,7 @@ case $cpu-$vendor in
>>>  			| hppa | hppa1.[01] | hppa2.0 | hppa2.0[nw] | hppa64 \
>>>  			| hexagon \
>>>  			| i370 | i*86 | i860 | i960 | ia16 | ia64 \
>>> +			| intelgt \
>>>  			| ip2k | iq2000 \
>>>  			| k1om \
>>>  			| kvx \
>>>
>>
>> Patch 3 enables ld, but I can't spot any gas enabling (sadly the cover letter
>> wasn't Cc-ed to the binutils list).
> 
> Sorry, this is my bad.  I'll Cc binutils in the next revision submission.
> For the time being, the link to the cover letter is
> 
>   https://sourceware.org/pipermail/gdb-patches/2024-December/214029.html
> 
>> Don't you further need to exclude gas from
>> attempts of configuring?
>>
>> I'm also puzzled by that difference: The series supposedly is about enabling
>> gdb. Why enable ld there? Just because it's (seemingly) easy?
>>
>> Jan
> 
> Without enabling ld, when configured as
> 
>   <src>/configure --enable-targets="intelgt-elf"
> 
> 'make' gives
> 
>   *** ld does not support target intelgt-unknown-elf
>   *** see ld/configure.tgt for supported targets

Like I think I said for gas, that imo wants dealing with by a top level
configure change, adding ld to noconfigtargets for intelgt.

Jan