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

binja: optimize performance of name demangling #1610

Open
williballenthin opened this issue Jul 10, 2023 · 19 comments
Open

binja: optimize performance of name demangling #1610

williballenthin opened this issue Jul 10, 2023 · 19 comments
Labels
binary-ninja enhancement New feature or request help wanted Extra attention is needed

Comments

@williballenthin
Copy link
Collaborator

# TODO(xusheng): performance optimizations to improve test execution time

@williballenthin williballenthin added enhancement New feature or request help wanted Extra attention is needed labels Jul 10, 2023
@tehami02
Copy link

@williballenthin I want to work on this.

@williballenthin
Copy link
Collaborator Author

this issue requires a license of Binary Ninja in order to triage and develop. do you have one?

you'll also need to work with @xusheng6 to clearly define the problem and propose a fix. i recall chatting with him about an opportunity for better performance but i don't remember the details.

@tehami02
Copy link

@williballenthin No I dont have Binary Ninja License, maybe I should try some GFI's first.

@williballenthin
Copy link
Collaborator Author

i think #1607 and #1608 might be reasonable places to start.

@tehami02
Copy link

@williballenthin Thanks for reference, I will try them.

@tehami02

This comment was marked as resolved.

@mr-tz

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@xusheng6

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@xusheng6

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@xusheng6

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@xusheng6

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@tehami02

This comment was marked as resolved.

@williballenthin
Copy link
Collaborator Author

glad you got this figured out @tehami02!

im going to go back and hide a bunch of the comments above so that this thread appears on-topic and related to the name demangling issue. feel free to continue to ask questions and we're here to help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binary-ninja enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants