Skip to content

Skip failing test cases and threadlocalstorage fix #8020

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

Conversation

jasonmolenda
Copy link

No description provided.

bulbazord and others added 3 commits January 22, 2024 22:25
These tests are failing on CI. To unblock PRs, we are going to
investigate and fix these offline. They are being tracked internally at
Apple.

(cherry picked from commit 42f54a5)
The TLS implementation on apple platforms has changed. Instead of
invoking pthread_getspecific with a pthread_key_t, we instead perform a
virtual function call.

Note: Some versions of Apple's new linker do not emit debug symbols for
TLS symbols. This causes the TLS tests to fail because LLDB and dsymutil
expects there to be debug symbols to resolve the relevant TLS block. You
may work around this by switching to the older linker (ld-classic) or by
disabling the TLS tests until you have a newer version of the new
linker.

rdar://120676969
(cherry picked from commit a451c3b)
(cherry picked from commit c0776cc)
The new static linker in Xcode 15 does not emit the necessary
symbols for file static thread local storage, causing this test
to fail when used.  The old static linker is still available
as ld-classic in Xcode 15, but it has to be invoked specially, and
the new static linker will be fixed at some point.  I may try to
add linker name and versioning information in
lldb/packages/Python/lldbsuite/test/decorators.py like we do with
the compiler / compiler_version, so it can be xfailed for known
problematic static linker name / versions, but until I get that
sorted I'm skipping this test to unblock the CI bots.

(cherry picked from commit bd3838f)
@jasonmolenda
Copy link
Author

@swift-ci test

The test is failing on x86_64 but passing on arm64. Skip until Ismail
can investigate this further.

(cherry picked from commit c0d1128)
@jasonmolenda
Copy link
Author

@swift-ci test

@JDevlieghere
Copy link

@swift-ci test windows

1 similar comment
@jasonmolenda
Copy link
Author

@swift-ci test windows

@JDevlieghere JDevlieghere merged commit 037945d into swiftlang:swift/release/5.10 Jan 23, 2024
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