mbox series

[0/3,2.44?] aarch64: Assorted dependency fixes

Message ID 8f330e1b-7f31-0feb-7746-86ee519a74e7@e124511.cambridge.arm.com
Headers
Series aarch64: Assorted dependency fixes |

Message

Alice Carlotti Jan. 31, 2025, 1:16 p.m. UTC 
  All of these patches remove dependencies between features, which makes gas more
permissive for feature additions and more restrictive for feature removals.

Patches 1 and 2 address feature flags that were added in Binutils 2.42 and 2.43
respectively, and have not yet been released in GCC (I intend to amend the
dependencies there before the release of GCC 15).  I see no good reason to
specify the explicit removal of these features in a target string, so these
patches make gas more restrictive in all realistic scenarios.

Patch 3 makes gas strictly more permissive, and in particular enables support
for the target string  `-march=armv9-a+sme+nosve`.

Are these patches ok for master?  Additionally, if Nick and (a) Richard
approve, how about the 2.44 branch?

Comments

Richard Earnshaw (foss) Jan. 31, 2025, 2:55 p.m. UTC | #1 
On 31/01/2025 13:16, Andrew Carlotti wrote:
> All of these patches remove dependencies between features, which makes gas more
> permissive for feature additions and more restrictive for feature removals.
> 
> Patches 1 and 2 address feature flags that were added in Binutils 2.42 and 2.43
> respectively, and have not yet been released in GCC (I intend to amend the
> dependencies there before the release of GCC 15).  I see no good reason to
> specify the explicit removal of these features in a target string, so these
> patches make gas more restrictive in all realistic scenarios.
> 
> Patch 3 makes gas strictly more permissive, and in particular enables support
> for the target string  `-march=armv9-a+sme+nosve`.
> 
> Are these patches ok for master?  Additionally, if Nick and (a) Richard
> approve, how about the 2.44 branch?

These are ok (including backporting to 2.44).

R.
Alice Carlotti Jan. 31, 2025, 3:21 p.m. UTC | #2 
On Fri, Jan 31, 2025 at 02:55:16PM +0000, Richard Earnshaw (lists) wrote:
> On 31/01/2025 13:16, Andrew Carlotti wrote:
> > All of these patches remove dependencies between features, which makes gas more
> > permissive for feature additions and more restrictive for feature removals.
> > 
> > Patches 1 and 2 address feature flags that were added in Binutils 2.42 and 2.43
> > respectively, and have not yet been released in GCC (I intend to amend the
> > dependencies there before the release of GCC 15).  I see no good reason to
> > specify the explicit removal of these features in a target string, so these
> > patches make gas more restrictive in all realistic scenarios.
> > 
> > Patch 3 makes gas strictly more permissive, and in particular enables support
> > for the target string  `-march=armv9-a+sme+nosve`.
> > 
> > Are these patches ok for master?  Additionally, if Nick and (a) Richard
> > approve, how about the 2.44 branch?
> 
> These are ok (including backporting to 2.44).
> 
> R.

Thanks, I've pushed this to master.

Nick, is this ok for the release branch as well?
Nick Clifton Feb. 3, 2025, 3:23 p.m. UTC | #3 
Hi Andrew,

> Nick, is this ok for the release branch as well?

Sure, although I think that I was too slow in responding to this email, sorry.

Cheers
   Nick