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

Mt ctf #163

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

MatthewKhouzam
Copy link
Contributor

This is to share the work, not yet for merge

With larger enums, this can provide a significant performance boost

[Changed] Lazy format enums in CTF

Change-Id: Icb85752a6b05cf2df0bef9ac322582035c1ff029
Signed-off-by: Matthew Khouzam <[email protected]>
Change-Id: I4f4b8960fd9933c80d182c80548fc4cc66e06ce8
Signed-off-by: Matthew Khouzam <[email protected]>
Speeds up trace reading by ~5-10%. The concurrent hashmap was used to avoid
contention. But the function is stable A will always produce B. So we can use
a normal hashmap.

[Changed] speed up ctf parsing of lexical scopes

Change-Id: I75f4b68de08c43f8ccb11a6fb6cc19c2e7d02f1e
Signed-off-by: Matthew Khouzam <[email protected]>
This accelerates trace reading when you have more streams, it appears
to peak in performance when you have 2n streams where n is the number of
cpus.

Change-Id: I621d98ff18506a1b3e5c66a553b7464d7f28af0f
Signed-off-by: Matthew Khouzam <[email protected]>
Speeds up to O(1) for most cases

Change-Id: Ib5a634f19cbf5ec669d37bfd497cc2252964bf72
Signed-off-by: Matthew Khouzam <[email protected]>
@MatthewKhouzam
Copy link
Contributor Author

don't review, check it out ;)

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.

1 participant