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.

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

Add debug info and remove stripping #424

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Add debug info and remove stripping #424

merged 1 commit into from
Jun 21, 2024

Conversation

dudoslav
Copy link
Collaborator

This PR should add additional debug information:

  1. Remove binary stripping
  2. Change build type to RelWithDebInfo

@dudoslav dudoslav self-assigned this Jun 20, 2024
@dudoslav
Copy link
Collaborator Author

base) taco:vector_search dudoslav$ objdump -h _tiledbvspy.cpython-310-x86_64-linux-gnu.so | grep debug
 27 .debug_aranges 00018ff0  0000000000000000  0000000000000000  006ee25b  2**0
 28 .debug_info   0215f43e  0000000000000000  0000000000000000  0070724b  2**0
 29 .debug_abbrev 0000ba35  0000000000000000  0000000000000000  02866689  2**0
 30 .debug_line   0060d156  0000000000000000  0000000000000000  028720be  2**0
 31 .debug_str    02ef5fba  0000000000000000  0000000000000000  02e7f214  2**0
 32 .debug_line_str 000020c1  0000000000000000  0000000000000000  05d751ce  2**0
 33 .debug_loclists 00d6702c  0000000000000000  0000000000000000  05d7728f  2**0
 34 .debug_rnglists 0026651f  0000000000000000  0000000000000000  06ade2bb  2**0

and:

(base) taco:vector_search dudoslav$ file _tiledbvspy.cpython-310-x86_64-linux-gnu.so
_tiledbvspy.cpython-310-x86_64-linux-gnu.so: ELF 64-bit LSB shared object, x86-64, version 1 (GNU/Linux), dynamically linked, BuildID[sha1]=f0582f2ec9c3b5652afa951a2326eaa405505df8, with debug_info, not stripped

@dudoslav dudoslav merged commit 5c8fcf9 into main Jun 21, 2024
6 checks passed
@dudoslav dudoslav deleted the db/debub_info branch June 21, 2024 07:34
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