Skip to content

[lldb] Calculate process address if file address + section fails #8849

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

Conversation

augusto2112
Copy link

If the resulting address from the Address constructor that takes a file address + offset fails, fallback to calculating the process address.

rdar://128481320

@augusto2112
Copy link
Author

@swift-ci test

@augusto2112 augusto2112 force-pushed the proc-address-lldb-mem-reader branch from b0b2747 to b642c3c Compare June 5, 2024 18:29
@augusto2112
Copy link
Author

@swift-ci test

@augusto2112 augusto2112 force-pushed the proc-address-lldb-mem-reader branch from b642c3c to 370bab6 Compare June 6, 2024 23:10
@kastiglione
Copy link

Is it possible to write a test for this?

@augusto2112 augusto2112 force-pushed the proc-address-lldb-mem-reader branch from 370bab6 to bd503d4 Compare June 6, 2024 23:59
If the resulting address from the Address constructor that takes a
file address + offset fails, fallback to calculating the process
address.

rdar://128481320
@augusto2112 augusto2112 force-pushed the proc-address-lldb-mem-reader branch from bd503d4 to 1b147a2 Compare June 7, 2024 00:01
@augusto2112
Copy link
Author

Is it possible to write a test for this?

Unfortunately I don't think that this is easily testable.

@augusto2112
Copy link
Author

@swift-ci test

@adrian-prantl adrian-prantl merged commit de395d3 into swiftlang:swift/release/6.0 Jun 18, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants