-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
sys::thread_local::key::tests::destructors fails under x86 Windows 7 #141300
Copy link
Copy link
Closed
Labels
A-atomicArea: Atomics, barriers, and sync primitivesArea: Atomics, barriers, and sync primitivesA-destructorsArea: Destructors (`Drop`, …)Area: Destructors (`Drop`, …)A-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcA-thread-localsArea: Thread local storage (TLS)Area: Thread local storage (TLS)C-bugCategory: This is a bug.Category: This is a bug.O-windows-7OS: Windows 7 or Windows Server 2008 R2 or etc.OS: Windows 7 or Windows Server 2008 R2 or etc.O-windows-msvcToolchain: MSVC, Operating system: WindowsToolchain: MSVC, Operating system: WindowsO-x86_32Target: x86 processors, 32 bit (like i686-*) (also known as IA-32, i386, i586, i686)Target: x86 processors, 32 bit (like i686-*) (also known as IA-32, i386, i586, i686)P-lowLow priorityLow priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.Relevant to the library team, which will review and decide on the PR/issue.regression-from-stable-to-betaPerformance or correctness regression from stable to beta.Performance or correctness regression from stable to beta.
Metadata
Metadata
Assignees
Labels
A-atomicArea: Atomics, barriers, and sync primitivesArea: Atomics, barriers, and sync primitivesA-destructorsArea: Destructors (`Drop`, …)Area: Destructors (`Drop`, …)A-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcA-thread-localsArea: Thread local storage (TLS)Area: Thread local storage (TLS)C-bugCategory: This is a bug.Category: This is a bug.O-windows-7OS: Windows 7 or Windows Server 2008 R2 or etc.OS: Windows 7 or Windows Server 2008 R2 or etc.O-windows-msvcToolchain: MSVC, Operating system: WindowsToolchain: MSVC, Operating system: WindowsO-x86_32Target: x86 processors, 32 bit (like i686-*) (also known as IA-32, i386, i586, i686)Target: x86 processors, 32 bit (like i686-*) (also known as IA-32, i386, i586, i686)P-lowLow priorityLow priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.Relevant to the library team, which will review and decide on the PR/issue.regression-from-stable-to-betaPerformance or correctness regression from stable to beta.Performance or correctness regression from stable to beta.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Code
Running the
sys::thread_local::key::tests::destructorstest under x86 Windows 7 (i686-win7-windows-msvc) currently fails with the following error:Version it worked on
The test does not fail when using
dev@ 3c5aef3, or a7c39b6 if only considering first-parentmastermerge commits.Version with regression
It starts to fail with the following 67a97ba commit of the same 077cedc rollup. It still fails more recently with 2b96ddc.
This is therefore definitely caused by #140007. That makes sense considering there hasn't been much other changes to the target recently. However, it is concerning because the target should still have emulated TLS, as intended by the PR "at fault". There could thus be a problem with the TLS emulation itself.
@rustbot label A-atomic A-thread-locals A-destructors A-testsuite O-x86_32 O-windows-7 O-windows-msvc
@rustbot modify labels: +regression-from-stable-to-beta -regression-untriaged