Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

@dotnet/arm64-contrib @amanasifkhalid

This is cleaner than the existing code, but it's a lot of code churn. The templates are just a each a one liner, then the same two column changes in all the *Tests.cs files and the proper code changes in the generator.

Because of the mechanical nature, and it's only the testing, this should be fairly low risk. But it's still a lot of code. I wonder if this should be left to .NET11?

Because of the mechanical nature, and it's only the testing, this should be fairly low risk. But it's still a lot of code. I wonder if this should be left to .NET11?

Unless anyone else has cold feet, I'm fine with taking this in .NET 10. @jonathandavies-arm could you PTAL at the merge conflicts? Once we get a CI run in, it should be easy to see if this regresses anything.

Because of the mechanical nature, and it's only the testing, this should be fairly low risk. But it's still a lot of code. I wonder if this should be left to .NET11?

Unless anyone else has cold feet, I'm fine with taking this in .NET 10. @jonathandavies-arm could you PTAL at the merge conflicts? Once we get a CI run in, it should be easy to see if this regresses anything.

Rebasing this is a pain and also is a pain for anyone with WIP to rebase on top. Given we're hitting .NET10 window, I suggest in order we:

1. Get SVE2 FP API's #118332 merged
2. @jacob-crawley raises a new PR to implement the missing FP APIs
3. That'll take us more or less up to the August 15th deadline
4. @jonathandavies-arm can rebase this PR, and then get it merged
5. @jacob-crawley will be looking at scatterstores/gatherloads, which can go last

Rebasing this is a pain and also is a pain for anyone with WIP to rebase on top. Given we're hitting .NET10 window, I suggest in order we:

Agreed, sounds like a good plan

Sorry for the force push but merging was confusing.

@jonathandavies-arm, do you need help with resolving conflicts?

Sorry for the force push but merging was confusing.

#120291 unblocks this PR.
@jonathandavies-arm, please resolve conflicts and ping us when review is ready.

#120291 unblocks this PR. @jonathandavies-arm, please resolve conflicts and ping us when review is ready.

@JulieLeeMSFT #118957 is also waiting to be merged and changes Sve2Tests.cs. I will start resolving the conflicts but I'll wait for this other PR before pushing it.

@JulieLeeMSFT #118957 is also waiting to be merged and changes Sve2Tests.cs. I will start resolving the conflicts but I'll wait for this other PR before pushing it.

Thanks for pointing it out. Asked Tanner to review #118957. Is it all you need?

@JulieLeeMSFT #118957 is also waiting to be merged and changes Sve2Tests.cs. I will start resolving the conflicts but I'll wait for this other PR before pushing it.

Thanks for pointing it out. Asked Tanner to review #118957. Is it all you need?

Pending on #118957, where the next step is to rerun its CI tests.

I am closing this PR and it is being replaced by #121584. It would be too much work to get his branch up to date with the latest changes on main without force pushing. There shouldn't be any conflicts with other PRs atm so hopefully it can be merged soon.