Skip to content

[RemoteInspection] Fix rebuildStorageTypeInfo for unmanaged types#88770

Merged
augusto2112 merged 1 commit intoswiftlang:mainfrom
augusto2112:fix-unamanged-pointer
May 5, 2026
Merged

[RemoteInspection] Fix rebuildStorageTypeInfo for unmanaged types#88770
augusto2112 merged 1 commit intoswiftlang:mainfrom
augusto2112:fix-unamanged-pointer

Conversation

@augusto2112
Copy link
Copy Markdown
Contributor

rebuildStorageTypeInfo only handled ReferenceTypeInfo and RecordKind::ClassExistential inner types for unmanaged_storage, but failed for imported unmanaged types which are imported behind a pointer (which is a struct).

rdar://175379963

rebuildStorageTypeInfo only handled ReferenceTypeInfo and
RecordKind::ClassExistential inner types for unmanaged_storage, but
failed for imported unmanaged types which are imported behind a
pointer (which is a struct).

rdar://175379963
@augusto2112 augusto2112 requested a review from a team as a code owner April 30, 2026 23:11
augusto2112 added a commit to augusto2112/llvm-project that referenced this pull request Apr 30, 2026
Exercise the fix for swiftlang/swift#88770
in both configurations: a Clang-bridged opaque pointer type
(MyBridgedRef), and a Foundation CF type (CFError).

Assisted-by: claude
@augusto2112
Copy link
Copy Markdown
Contributor Author

@swift-ci smoke test

@augusto2112
Copy link
Copy Markdown
Contributor Author

@swift-ci smoke test Windows

@augusto2112 augusto2112 requested a review from mikeash May 1, 2026 16:27
@augusto2112 augusto2112 merged commit c0da3a8 into swiftlang:main May 5, 2026
3 checks passed
augusto2112 added a commit to augusto2112/llvm-project that referenced this pull request May 6, 2026
Exercise the fix for swiftlang/swift#88770
in both configurations: a Clang-bridged opaque pointer type
(MyBridgedRef), and a Foundation CF type (CFError).

Assisted-by: claude
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.

3 participants