-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Bump lldb-mi commit hash #11566
base: main
Are you sure you want to change the base?
Bump lldb-mi commit hash #11566
Conversation
This alone wont upgrade the version of lldb-mi used. Someone will need to re-run the internal lldb-mi and update the release. |
Will you be able to do that @WardenGnaw? |
Theres a couple issues that I do not have the cycles to fix.
@ylatuya Were you able to build lldb-mi locally with LLVM commit |
To fix the lldb-mi bug, I built it using main, which at that time was at commit:
|
@ylatuya Sorry for off-topic do you know how to fix this issue or how to approach fixing it: lldb-tools/lldb-mi#101 |
I don't know how to fix it, sorry. This was my first contribution and it was a trivial fix. |
@ylatuya There seems to be a proposed fix for the memory leak issue. Could you please update this PR to the latest commit: a6c8c66 here: https://github.com/lldb-tools/lldb-mi/commits/main/ @WardenGnaw @bobbrow Is there any change updating this PR and moving it forward? |
Fix #6874