Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Leak sanitizer UI test hangs on Macos ARM64 #88132

Open
hkratz opened this issue Aug 18, 2021 · 5 comments
Open

Leak sanitizer UI test hangs on Macos ARM64 #88132

hkratz opened this issue Aug 18, 2021 · 5 comments
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-sanitizers Area: Sanitizers for correctness and code quality C-bug Category: This is a bug. O-AArch64 Armv8-A or later processors in AArch64 mode O-macos Operating system: macOS T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@hkratz
Copy link
Contributor

hkratz commented Aug 18, 2021

Try on a Apple Silicon Mac:

./configure --build=aarch64-apple-darwin --enable-sanitizers --enable-profiler --set rust.jemalloc --set llvm.ninja=false
./x.py test --stage 1  src/test/ui/sanitize/leak.rs

Expected result: Test passes.

Actual result: Test hangs:

running 1 test
test [ui] ui/sanitize/leak.rs has been running for over 60 seconds

See also: #88131

@rustbot label +A-sanitizer +A-llvm +O-macos +O-arm
@hkratz hkratz added the C-bug Category: This is a bug. label Aug 18, 2021
@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. O-Arm Target: 32-bit Arm processors (armv6, armv7, thumb...), including 64-bit Arm in AArch32 state O-macos Operating system: macOS labels Aug 18, 2021
@hkratz
Copy link
Contributor Author

hkratz commented Aug 18, 2021

This problem is still reproducible with LLVM 13.

hkratz added a commit to rusticstuff/rust that referenced this issue Sep 22, 2021
@hkratz
Disable the leak sanitizer on Macos aarch64 for now.
59e37c8 
It is currently broken, see rust-lang#88132.
@hkratz hkratz mentioned this issue Sep 22, 2021
Merged
the8472 added a commit to the8472/rust that referenced this issue Sep 22, 2021
@the8472
Rollup merge of rust-lang#89170 - rusticstuff:aarch64_macos_disable_l…
26c7838 
…eak_sanitizer, r=petrochenkov

Disable the leak sanitizer on Macos aarch64 for now

It is currently broken, see rust-lang#88132.
@workingjubilee workingjubilee added O-AArch64 Armv8-A or later processors in AArch64 mode and removed O-Arm Target: 32-bit Arm processors (armv6, armv7, thumb...), including 64-bit Arm in AArch32 state labels Mar 18, 2022
@csmoe csmoe mentioned this issue Jun 26, 2022
Open
@Noratrieb Noratrieb added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Apr 5, 2023
@Urgau Urgau mentioned this issue Feb 26, 2024
Open
@Jules-Bertholet
Copy link
Contributor

@rustbot label A-sanitizers

@rustbot rustbot added the A-sanitizers Area: Sanitizers for correctness and code quality label Apr 8, 2024
@Coder-256
Copy link
Contributor

The test seems to pass now (tested with ./x.py test --stage 1 tests/ui/sanitizer/leak.rs), would it be reasonable to consider re-enabling leak sanitizer on aarch64-apple-darwin now?

@ehuss
Copy link
Contributor

ehuss commented Oct 24, 2024

For context, leak sanitizer was disabled in #89170.

@madsmtm
Copy link
Contributor

madsmtm commented Nov 13, 2024

I think this issue can be closed (using the sanitizer no longer hangs), but #121624 is still a blocker before we can enable it as supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-sanitizers Area: Sanitizers for correctness and code quality C-bug Category: This is a bug. O-AArch64 Armv8-A or later processors in AArch64 mode O-macos Operating system: macOS T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@ehuss @hkratz @Coder-256 @madsmtm @workingjubilee @rustbot @Noratrieb @Jules-Bertholet