Swift 4.1 branch

[atrick]

Merge bug fixes into 4.1.

[atrick]

@swift-ci smoke test.

[atrick]

@eeckstein is this the correct way to cherry pick into 4.1, and if so can you approve it so I can merge the PR?

[eeckstein]

MandatoryInlining changes LGTM.
The IRGen changes should be reviewed by someone else

[atrick]

I'll wait for @shajrawi to review the LoadableAddress changes.

[shajrawi]

LGTM

[atrick]

@swift-ci smoke test and merge.

[jrose-apple]

Small request: can you title your 4.1 PRs more clearly in the future? I'd like to know what you're doing to the 4.1 branch without having to click into the PR.

[atrick]

@shahmishal This PR fixes the bots that were already broken at the time the 4.1 branch was cut. I don't know how to get this merged. Is it something you can do?

[gparker42]

This is a functional change, so you'll need CCC review for the 4.1 branch.

[atrick]

@swift-ci nominate.

This does not really need CCC. They are the "emergency" bug fixes for bots that were broken at the time the branch was cut and are completely obvious. I originally didn't intend for anyone else to even look at this PR. But since it took so long for the PR to be merged, we now have an official process. So here goes:

Explanation: Same-day fixes for two broken bots at the time the 4.1 branch was cut.

Scope: ASAN and source-compatibility tests.

Origin: d369aa4 Support @NoEscape SIL function types. (#12420)

Radar:
rdar://problem/35059942 Failed - Lark, 3.0, Swift Package - Source Compatibility Suite (master)
rdar://problem/35055251 FAILED: ASAN use-after-free in MandatoryInlining.

Risk: These are minimally invasive surgical fixes.

Testing: See scope.

[shahmishal]

@atrick swift-4.1-branch requires full testing, because its under CCC.

@swift-ci test

[shahmishal]

@swift-ci test source compatibility