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

Label feature compiler #476

Closed

Conversation

vi3tL0u1s
Copy link
Contributor

  • Implement compilable label statement functionality
  • Extend Continue statement to accept label string value
  • Add label context tracking in Context analyzer

Performance Analysis:

  • Conducted 2-hour comparison test on V8
  • Test environment: 16GB RAM, 8 Core CPU VM
  • Results:
    • 1% improvement in all coverage metrics (functions, lines, regions, branches)
    • Enhanced coverage in 252 source files

This implementation builds upon the label support work from PR #467,
adding the compilation capability.

Yi2255 and others added 2 commits November 14, 2024 12:00
- Implement compilable label statement functionality
- Extend Continue and Break statements to accept label string value
- Add label context tracking in Context analyzer
- Improve overall compiler handling of labeled statements

Performance Analysis:
- Conducted 2-hour comparison test on V8
- Test environment: 16GB RAM, 8 Core CPU VM
- Results:
  * 1% improvement in all coverage metrics (functions, lines, regions, branches)
  * Enhanced coverage in 252 source files

This implementation builds upon the label support work from PR #467,
adding compilation capability.
Copy link

google-cla bot commented Nov 25, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@saelo saelo mentioned this pull request Nov 26, 2024
@vi3tL0u1s
Copy link
Contributor Author

Hi @saelo, I will close this PR for the inconvenience! I asked @Yi2255 and am allowed to add a commit into the existing #467 PR!

@vi3tL0u1s vi3tL0u1s closed this Nov 27, 2024
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.

2 participants