Kugan Vivekanandarajah <kvivekana...@nvidia.com> writes:
> Adding Eugene and Andi to CC as Sam suggested.
>
>> On 13 May 2025, at 12:57 am, Richard Sandiford <richard.sandif...@arm.com> 
>> wrote:
>>
>> External email: Use caution opening links or attachments
>>
>>
>> Kugan Vivekanandarajah <kvivekana...@nvidia.com> writes:
>>> diff --git a/configure.ac b/configure.ac
>>> index 730db3c1402..701284e38f2 100644
>>> --- a/configure.ac
>>> +++ b/configure.ac
>>> @@ -621,6 +621,14 @@ case "${target}" in
>>>     ;;
>>> esac
>>>
>>> +autofdo_target="i386"
>>> +case "${target}" in
>>> +  aarch64-*-*)
>>> +    autofdo_target="aarch64"
>>> +    ;;
>>> +esac
>>> +AC_SUBST(autofdo_target)
>>> +
>>> # Disable libssp for some systems.
>>> case "${target}" in
>>>   avr-*-*)
>>
>> Couldn't we use the existing $cpu_type, rather than adding a new variable?
>> I don't think the two would ever need to diverge.
>
> I tried doing this but looks to me that $cpu_type is available only in 
> libgcc. Am I missing something  or do you want me to replicate that here?

What I meant was that we could simply add:

AC_SUBST(cpu_type)

rather than add a new variable and case statement.

Thanks,
Richard

Reply via email to