Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Print opcode count and total size in debug output #929

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

bnoordhuis
Copy link
Contributor

Fixes: #777

@bnoordhuis bnoordhuis merged commit 12f7c55 into quickjs-ng:master Feb 25, 2025
59 checks passed
@bnoordhuis bnoordhuis deleted the fix777 branch February 25, 2025 10:44
@gengjiawen
Copy link

This lacks overall bytecode. looks now based on function

@bnoordhuis
Copy link
Contributor Author

Sum them? Printing the total would be redundant, the information is already there.

@gengjiawen
Copy link

Sum them? Printing the total would be redundant, the information is already there.

Without this large code will hard to measure perf, like compare bytecode size to v8. I will share more details tomorrow since it’s very late in my timezone.

@gengjiawen
Copy link

http://k2.gengjiawen.com:8001/diff

I made a diff page

@gengjiawen
Copy link

Sum them? Printing the total would be redundant, the information is already there.

I found out v8 is split too. So current behavior the same with v8, I will sum it with extra method.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

expose the bytecode in cli
3 participants