[SR-7100] Change default ownership for passing parameters to use the "guaranteed" convention

[eeckstein]

Previous ID	SR-7100
Radar	rdar://problem/34120147
Original Reporter	@eeckstein
Type	New Feature
Status	Resolved
Resolution	Done

Additional Detail from JIRA

Votes	0
Component/s	Compiler
Labels	New Feature
Assignee	@gottesmm
Priority	Medium

md5: 161a7e413f6f94d7eea269d013807334

relates to:

• SR-4355 Update standard library to use memory ownership features

Issue Description:

For details see: https://forums.swift.org/t/proposal-to-change-the-default-ownership-for-passing-parameters/10261

[gottesmm]

I am starting to "thin" the branch out by upstreaming things. I am going to turn on +0 with a small commit so I can revert. This is going to cause me to need to upstream all the test updates/etc before that. The hope is to merge the final version of this after the stable-merge happens tomorrow.

[gottesmm]

Branch thinning is continuing from ~30 patches to ~20.

[gottesmm]

Down to ~5.

[gottesmm]

Just to update this real quick:

1. On the bots I got my first srouce compatibility success run.
2. There was some confusion around how I was going to upstream this. I am no longer doing the nice way that would let us revert. It was taking me too long to get the tooling correct (I kept on running into problems caused by moving tests). I am now just going the old traditional way of change the world.

[gottesmm]

Another update. The benchmarks all pass except for floating point tests that are also failing on master without +0. Making progress.

[gottesmm]

Floating point tests fixed on master. Reran benchmarks everything works. Fixing the last couple of failing tests and then we should have a completely green PR in a bit.

[gottesmm]

Finally got this through the CI in: 54b38d4

[bob-wilson]

Sounds like this is done then. Congrats, Michael!

[gottesmm]

Yes it is complete!