swift
Resolve recently introduced warnings
#76349
Merged

Resolve recently introduced warnings #76349

tshortli
tshortli337 days ago

Resolve various warnings that were introduced recently into the compiler and stdlib builds. See individual commits for specific justifications.

tshortli Concurrency: Remove superflous _SwiftConcurrencyShims imports.
ba3cd79b
tshortli Sema: Refactor @_implementationOnly deprecation warning suppression.
99fd29c3
tshortli Sema: Allow some @_implementationOnly imports in stdlib modules.
2ec8efea
tshortli AST: Suppress unused function and -Wreturn-type-c-linkage warnings.
896ed660
tshortli Backtracing: Use internal import in swift-backtracing.
28bc3ca8
tshortli tshortli requested a review from xymus xymus 337 days ago
tshortli tshortli requested a review from DougGregor DougGregor 337 days ago
tshortli tshortli requested a review from al45tair al45tair 337 days ago
tshortli tshortli requested a review from ktoso ktoso 337 days ago
tshortli tshortli requested a review from hborla hborla 337 days ago
tshortli tshortli requested a review from slavapestov slavapestov 337 days ago
tshortli tshortli requested a review from xedin xedin 337 days ago
tshortli
tshortli337 days ago

@swift-ci please smoke test

tshortli
tshortli337 days ago

@swift-ci please build toolchain Linux platform

tshortli
tshortli337 days ago

@swift-ci please build toolchain Windows platform

tshortli
tshortli337 days ago

@swift-ci please build toolchain macOS platform

DougGregor
DougGregor approved these changes on 2024-09-09
DougGregor337 days ago

Looks great, thank you!

lib/AST/InlinableText.cpp
196#pragma clang diagnostic push
197#pragma clang diagnostic ignored "-Wreturn-type-c-linkage"
198extern "C" BridgedStringRef
199
swift_ASTGen_extractInlinableText(BridgedASTContext ctx,
DougGregor337 days ago

Thank you!

xymus
xymus approved these changes on 2024-09-09
tshortli
tshortli337 days ago

The macOS toolchain build is broken by - error: no template named 'span' in namespace 'std' which is unrelated to this change.

tshortli tshortli merged 60f049d9 into main 337 days ago
tshortli tshortli deleted the resolve-warnings branch 337 days ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone