Skip to content

Manually merged upstream commits from apple/swift:master #33943

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1,087 commits into from
Sep 15, 2020

Conversation

dabrahams
Copy link
Contributor

No description provided.

hborla and others added 30 commits September 4, 2020 12:34
…test

[Test] Disable Sanitizers/symbolication.swift
[test][IRGen] Loosen check in test unused.sil.
…xtern

[IRGen] Use the lexical decl context for VarDecls.
A bunch of callers were accidentally misusing it.
Refactor TypeCheckFunctionBodyRequest to return
the type-checked body, and remove
`typeCheckAbstractFunctionBody` in favor of
a method on AbstractFunctionDecl. In addition,
start returning an ErrorExpr body instead of
a partially type-checked body if type-checking
fails.
We no longer run error checking for already
type-checked synthesized functions, so add a
couple of `setThrows` calls where they were
previously missing.
Use `getTypecheckedBody` to allow lazy
type-checking when emitting function definitions.
…version is invalid

If the right-hand side (destination) of value-to-pointer conversion
is incorrect e.g. base type of member is a hole, let's record
a generic "invalid conversion" failure.

Resolves: rdar://problem/68254165
Fix the parsing of await/try/try?/try! so that the two can be nested
(e.g., `await try f()` or `try await f()`).

Then, fix the effects checking logic to preserve all throws-related
information under an `await`, and preserve all async/await-related
information under `try`/`try?`/`try!`.

Add a number of tests, and fix 'await' handling for string
interpolations as well.
utils: add a timeout to the integration tests
…sions

ModuleInterface: remark potential version differences between SDK and prebuilt modules
…-await-try

[Concurrency] Fix nested await/try parsing and effects checking.
…mance

[benchmark] Add new benchmark for floating-point conversion
…-windows-rebranch

Delete build-windows-rebranch.bat
…scope-lookup-flag

Remove -enable-astscope-lookup flag
…on-inlined locs.

The reason why is that due to ARCCodeMotion, they could have moved enough that
the SourceLoc on them is incorrect. That is why you can see in the tests that I
had to update, I am moving the retain to the return statement from the body of
the function since the retain was now right before the return.

I also went in and cleaned up the logic here a little bit. What we do now is
that we have a notion of instructions that we /always/ infer SourceLocs for (rr)
and ones that if we have a valid non-inlined location we use (e.x.:
allocations).

This mucked a little bit with my ability to run SIL tests since the SIL tests
were relying on this not happening to rr so that we would emit remarks on the rr
instructions themselves. I added an option that disables the always infer
behavior for this test.

That being said at this point to me it seems like the SourceLoc inference stuff
is really tied to OptRemarkGenerator and I am going to see if I can move it to
there. But that is for a future commit on another day.
Windows IR seems to include a `dso_local` annotation, which the check
didn't expect and caused a failure in swiftlang#33813. Added a regex check to
ensure the test passes with or without the annotation.
[ConstraintSystem] Record generic fix if destination of a pointer con…
The WinSDK module failed to expose WinBase properly for reuse, which the
newer clang seems to object to (via an assertion).  Add a separate
module for WinBase which allows reuse of the definitions without causing
conflicts.  This caused some additional fallout requiring the creation
of the WinSVC submodule (service handling, part of security and
identity) and splitting up the legacy WinSock header from the WinSock2
module.  This allows building Foundation again on Windows.
CodaFi and others added 20 commits September 11, 2020 22:28
Add a new constructor that takes an explicit file_type::ID to allow for the eventual deletion of InputFileKind.
Tying InputFile to this option meant that every input that was not one of the explictly-blessed kinds was modeled as a Swift file.

With the new InputFile that infers file kinds, we no longer need CompilerInvocation::setInputKind
SILParser: Fix for removal of parse-time name lookup
…tion

[sil-opt] Fix covering all-case of trapping instruction of SILInstruction::mayTrap
This flag is a GNU extension, and would cause misleading test failures
on other platforms where this extension is not available. However, the
necessity to switch line endings is only required on Windows when
testing. We could use sed to canonicalize line endings before comparing,
but that may cause higher amounts of filesystem traffic on Windows which
would slow down testing.

Instead, move the substitution for diff in SourceKit's lit.local.cfg up
to the top level, and conditionalize the substitution which has the flag
on Windows, but not on other platforms (where it should not be required).
ABI Checker: use lower-cased decl keywords to be consistent with source. NFC
* 'tensorflow' of github.com:apple/swift:
  [AutoDiff] Generate transparent ossa reabstraction thunks. (swiftlang#33897)
[test] diff --strip-trailing-cr is non-standard.
@texasmichelle
Copy link
Contributor

Please test with following PRs:
swiftlang/llvm-project#1803

@swift-ci Please clean test TensorFlow

@texasmichelle texasmichelle changed the base branch from tensorflow to tensorflow-merge September 15, 2020 18:16
@texasmichelle
Copy link
Contributor

Please test with following PRs:
swiftlang/llvm-project#1803

@swift-ci Please clean test TensorFlow

@texasmichelle
Copy link
Contributor

Merging since CI tests do not reflect changes in swiftlang/llvm-project#1803.

@texasmichelle texasmichelle merged commit 35604e9 into swiftlang:tensorflow-merge Sep 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.