Skip to content

Enable LLDB debug info tests on CI #141539

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EFanZh
Copy link
Contributor

@EFanZh EFanZh commented May 25, 2025

This PR tries to enable LLDB debug info tests on CI, so that unintentional changes to LLDB debug info can be detected.

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels May 25, 2025
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch 2 times, most recently from a80cafb to 9535bd7 Compare May 25, 2025 11:13
@rustbot rustbot added A-compiletest Area: The compiletest test runner T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels May 25, 2025
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch from 9535bd7 to d06f1df Compare May 25, 2025 12:40
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch from d06f1df to d036af0 Compare May 25, 2025 14:42
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch 3 times, most recently from f1ef40e to 048ec5e Compare May 25, 2025 16:40
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch from 048ec5e to 62d519c Compare May 26, 2025 05:13
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch from 62d519c to de5ae89 Compare May 26, 2025 06:30
@rust-log-analyzer

This comment has been minimized.

@EFanZh EFanZh force-pushed the enable-lldb-debug-info-tests-on-ci branch from de5ae89 to 573137d Compare May 26, 2025 07:05
@EFanZh
Copy link
Contributor Author

EFanZh commented May 26, 2025

I have observed that several tests were broken, which I marked NOT WORKING, and some pretty printing style has changed, not sure if they were intentional.

@EFanZh EFanZh marked this pull request as ready for review May 26, 2025 11:10
@rustbot
Copy link
Collaborator

rustbot commented May 26, 2025

r? @Kobzol

rustbot has assigned @Kobzol.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 26, 2025
@Kobzol
Copy link
Contributor

Kobzol commented May 26, 2025

CC @saethlin I remember you having thoughts on how we should run debugger tests on CI. Personally, I would love for our debugger support to improve, of course, but I'm not sure if we actually have someone working on debugging at the moment and if the debugger support is at a reasonable enough level for us to re-enable the tests.

@saethlin
Copy link
Member

saethlin commented May 26, 2025

The problem with the debuginfo tests is that they just run whatever is installed locally (the tracking issue for this problem is #126092). I ran into llvm/llvm-project#70453, which broke lldb for 2 major LLVM releases because upstream failed to backport the fix.

I have an overly negative view of the test suite because my primary interaction with it is discovering that it is completely broken. I'm aware that some people try to justify the current state of it, but I think that's just knee-jerk status quo defense and if someone tried to introduce this test suite today it would not get merged.

I don't actually work on debuginfo generation or Rust debugger support. As far as I'm aware, such work is primarily out-of-tree and mostly we just adjust our code or tests to match whatever upstream does. There is a Zulip stream #t-compiler/debuginfo but it has not had any new messages in the past 5 months.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-compiletest Area: The compiletest test runner A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants