-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
UI test async-drop-initial now fails size assertion on aarch64-unknown-none #140939
Copy link
Copy link
Closed
Closed
Copy link
Labels
A-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcC-bugCategory: This is a bug.Category: This is a bug.F-async_drop`#![feature(async_drop)]``#![feature(async_drop)]`O-AArch64Armv8-A or later processors in AArch64 modeArmv8-A or later processors in AArch64 modeT-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.
Metadata
Metadata
Assignees
Labels
A-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcC-bugCategory: This is a bug.Category: This is a bug.F-async_drop`#![feature(async_drop)]``#![feature(async_drop)]`O-AArch64Armv8-A or later processors in AArch64 modeArmv8-A or later processors in AArch64 modeT-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.
Type
Fields
Give feedbackNo fields configured for issues without a type.
We found this issue while integrating recent upstream changes into ferrocene.
The easiest way to reproduce the issue is with the following Cargo project.
Cargo.toml.cargo/config.tomlsrc/main.rsIf you run this on a x86_64 host then you get the value 168 which is what the test expects
If you compile to
aarch64-unknown-noneand run the program in QEMU you get the value 136.The difference might be that the
aarch64-unknown-nonetarget is apanic=abort(non-unwinding) target so the size of the coroutine is different due to the lack of drop flags or landing pads.Note that both targets have 64-bit pointer which is what the UI test uses as the only criteria to execute the assertion or not.
Were are not exactly sure which recent PR broke the test as it was passing before but it should be one of the PRs listed in ferrocene/ferrocene#1459 (comment)
Meta
rustc --version --verbose: