> On 22 Apr 2025, at 15:31, Tamar Christina <tamar.christ...@arm.com> wrote:
> 
>> -----Original Message-----
>> From: Richard Sandiford <richard.sandif...@arm.com>
>> Sent: Tuesday, April 22, 2025 2:28 PM
>> To: Tamar Christina <tamar.christ...@arm.com>
>> Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw <richard.earns...@arm.com>;
>> ktkac...@nvidia.com
>> Subject: Re: [PATCH] Document AArch64 changes for GCC 15
>> 
>> Tamar Christina <tamar.christ...@arm.com> writes:
>>>> -----Original Message-----
>>>> From: Richard Sandiford <richard.sandif...@arm.com>
>>>> Sent: Tuesday, April 22, 2025 1:31 PM
>>>> To: gcc-patches@gcc.gnu.org
>>>> Cc: Richard Earnshaw <richard.earns...@arm.com>; ktkac...@nvidia.com;
>>>> Tamar Christina <tamar.christ...@arm.com>
>>>> Subject: [PATCH] Document AArch64 changes for GCC 15
>>>> 
>>>> The list is structured as:
>>>> 
>>>> - new configurations
>>>> - command-line changes
>>>> - ACLE changes
>>>> - everything else
>>>> 
>>>> As usual, the list of new architectures, CPUs, and features is from a
>>>> purely mechanical trawl of the associated .def files.  I've identified
>>>> features by their architectural name to try to improve searchability.
>>>> Similarly, the list of ACLE changes includes the associated ACLE
>>>> feature macros, again to try to improve searchability.
>>>> 
>>>> The list summarises some of the target-specific optimisations because
>>>> it sounded like Tamar had received feedback that people found such
>>>> information interesting.
>>>> 
>>>> I've used the passive tense for most entries, to try to follow the
>>>> style used elsewhere.
>>>> 
>>>> We don't yet define __ARM_FEATURE_FAMINMAX, but I'll fix that
>>>> separately.
>>>> 
>>>> How does this look?  Anything I missed?
>>> 
>>> Thanks! Looks good! I think we're missing the ILP32 deprecation notice.
>>> But also crucially that the multilibs aren't build by default anymore.
>> 
>> I'd left ILP32 out since it's already mentioned in the Caveats section:
>> 
>>  <li>In the AArch64 port, support for ILP32 (<code>-mabi=ilp32</code>) has
>>    been deprecated and will be removed in a future release.
>>  </li>
>> 
>> But I agree that the removal of the multilibs makes it worth a second
>> mention here.  (And also in case people interested in AArch64 don't
>> read the general Caveats section.)
>> 
>> How about:
>> 
>>  <li>As noted above, support for ILP32 (<code>-mabi=ilp32</code>)
>>    has been deprecated and will be removed in a future release.
>>    <code>aarch64*-elf</code> targets no longer build the ILP32 multilibs.
>>  </li>
>> 
> 
> Perfect!
> 
> Thanks for writing it up,

> Tamar
> 

Looks fine to me too, thanks for doing this.
We can incrementally adjust in the future if needed.
Kyrill


>> Thanks,
>> Richard

Reply via email to