-
Notifications
You must be signed in to change notification settings - Fork 13.4k
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
base: master
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
a80cafb
to
9535bd7
Compare
This comment has been minimized.
This comment has been minimized.
9535bd7
to
d06f1df
Compare
This comment has been minimized.
This comment has been minimized.
d06f1df
to
d036af0
Compare
This comment has been minimized.
This comment has been minimized.
f1ef40e
to
048ec5e
Compare
This comment has been minimized.
This comment has been minimized.
048ec5e
to
62d519c
Compare
This comment has been minimized.
This comment has been minimized.
62d519c
to
de5ae89
Compare
This comment has been minimized.
This comment has been minimized.
de5ae89
to
573137d
Compare
I have observed that several tests were broken, which I marked |
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. |
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. |
This PR tries to enable LLDB debug info tests on CI, so that unintentional changes to LLDB debug info can be detected.