Skip to content
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

[SR-3445] Investigate deserialization bugs when deserializing a sibgen version of stdlib core #46033

Open
gottesmm opened this issue Dec 18, 2016 · 0 comments
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. compiler The Swift compiler in itself SILSerialization

Comments

@gottesmm
Copy link
Member

Previous ID SR-3445
Radar None
Original Reporter @gottesmm
Type Bug
Additional Detail from JIRA
Votes 0
Component/s Compiler
Labels Bug, SILSerialization
Assignee None
Priority Medium

md5: c2d37167a2f43016c576481db0ceb12a

Issue Description:

While doing work on ownership, I compiled a sibgen version of SwiftCore and noticed that sil-opt can not deserialize it due to I believe an error when deserializing a type.

We currently have tests that make sure that we can always deserialize the optimized standard library and the -Onone standard library, but we do not check if we can deserialize silgen.

We should create some sort of automation to check this and have someone investigate this.

I am merging in a commit that ensures that the targets for generating sib/sibgen files are created but not invoked by default. You should be able to use it. I would give you a hash, but it is currently merging. The PR is:

#6351

The repo hashes around where this was checked was:

clang 3180cd1 Merge remote-tracking branch 'origin/swift-3.1-branch' into stable
cmark 912d9d4 Merge pull request #1 from hughbe/patch-1
compiler-rt d235517 Merge remote-tracking branch 'origin/swift-3.1-branch' into stable
llbuild bf3b11a [Xcode] Disable a noisy warning.
lldb d93dd0f Adding a Swift specific fix that goes along with SVN revision 289233 where we may not always have a DeclVendor.
llvm 325df4d [ARM] Fix for 64-bit CAS expansion on ARM32 with -O0
swift 1b84d13 [semantic-sil] Reify the split in SILArgument in between function and block arguments via subclasses.
swift-corelibs-foundation 173e9ea Merge pull request #709 from phausler/cflite-import
swift-corelibs-libdispatch a055ddb Merge pull request #186 from dgrove-oss/SR-3002
swift-corelibs-xctest 1879b06 Merge pull request #181 from thaliproject/update-copyright
swift-integration-tests decef53 Merge pull request #13 from apple/remove-swift-update-test
swift-xcode-playground-support 6ee810b Resign ownership of PlaygroundLogger as I will be leaving Apple at the end of the month. Connor will be taking over.
swiftpm d683bca Merge pull request #841 from aciidb0mb3r/fix-xcode-test-popen

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. compiler The Swift compiler in itself SILSerialization
Projects
None yet
Development

No branches or pull requests

1 participant