forked from swiftlang/swift
-
Notifications
You must be signed in to change notification settings - Fork 30
Resolve conflicts with master #1422
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This uses `io.open` to allow `test_util` to open with the encoding and newline handling across python 2 and python 3. With this change, the test suite failures are within the single digits locally.
This is required to ensure that we encode the strings correctly when redirected. This allows the printing to succeed where it would fail previously due to encoding failures.
…put. Clients that use -print-target-info can avoid an extra frontend invocation by using this information.
Describe the backward-deployment libraries via a preprocessor-driven table. Macro-metaprogramming the two places in the code base---the driver and IRGen---to use this tabble to determine which backward-compatibility libraries to link against.
The driver and any other client that attempts to properly link Swift code need to know which compatibility libraries should be linked on a per-target basis. Vend that information as part of -print-target-info.
…f an enum case witness
Appease the python linter
This used to cause duplicated results in call signature completions. i.e.: AlertViewController(#^HERE^# // 2 x (coder: NSCoder) rdar://problem/65081358
…e-inference-enum-witness [AssociatedTypeInference] Strip 'self' parameter from function type of an enum case witness
test: make `test_util` more Python 3 friendly
…081358 [CodeCompletion] Remove redundant entries from possible callee analysis
[Build System] Support host target prefix in symbols package
The test is passing now--un-XFAIL it. rdar://problem/50648519
…ationwithin it by inverting if statements. NFCI.
Rename "emitters" to "cloners", for consistency: - `JVPEmitter` -> `JVPCloner` - `VJPEmitter` -> `VJPCloner` - `PullbackEmitter` -> `PullbackCloner` Improve `PullbackCloner` documentation. - Document previously undocumented methods. - Update outdated documentation. - For adjoint value accumulation helpers: rename "buffer access" occurrences to "address". Pullback generation no logner uses buffer accesses (`begin_apply`).
…unction has a noreturn result. Sometimes one just has a SILFunctionConvention instead of the underlying SILFunctionType (that the SILFunctionConvention contains). This just shims in that API onto the composition type.
…59ab5a5aec0599bf9c1f930
…get-info [Frontend] Improve -print-target-info
…974d93fbe5509171ea31bd0 [gardening] Extract out a lambda out of a loop and reduce some indentation within it by inverting if statements.
…merge # Conflicts: # test/Driver/print_target_info.swift
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.