[pull] swiftwasm from main #4044
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
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
…tringRef` This causes problem with the Windows build and also causes a deserialization crash on Linux.
This cleans up the workaround for VS2015 which hasn't been needed for quite some time now as the minimum requirements was VS2017. We are now looking to move to VS2019 as a minimum compiler version on Windows, and so clean this up. Fixes: SR-3422
Using the qualified name is subtly different from scoping the function. Use the latter, which correctly places the function into the namespace, to enable building with VS2022.
Revert [cxx-interop][libswift] Use `std::string` instead of `BridgedStringRef`
…ymbols. PublicCMOSymbols stores symbols which are made public by cross-module-optimizations. Those symbols are primarily stored in SILModule and eventually used by TBD generation and validation.
* rename the CrossModuleSerializationSetup pass to simply CrossModuleOptimization * remove the CMO specific serializer pass. Instead run the CrossModuleSerializationSetup pass directly before the standard serializer pass. * correctly handle shared functions (e.g. specializations) * refactoring
AST: make `RequirementMachine/PropertyUnification.cpp` build on VS2022
Reflections: remove some VS2015 support code
rework cross-module-optimization
…ftlang#40610) This allows the file to be easily included where needed (e.g. `StandaloneOverlay.cmake`) and reduce the likelihood of miscompilation due to missing sensible defaults. As a start, focus on a handful of parameters that got added/modified in recent PRs. Addresses rdar://85978195
MaxDesiatov
approved these changes
Dec 20, 2021
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.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )