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

Jan Beulich
2024-12-18T07:26:13Z

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