Skip to content

feat: simplify inspector interpreter loop#2731

Closed
rakita wants to merge 4 commits intomainfrom
rakita/insp-simple-end
Closed

feat: simplify inspector interpreter loop#2731
rakita wants to merge 4 commits intomainfrom
rakita/insp-simple-end

Conversation

@rakita
Copy link
Member

@rakita rakita commented Jul 18, 2025

Removes additional check for loop end in inspector interpreter loop. Comparing it with older this looked even worse

Will merge inspector on main first #2730 so it can be compared more easily.

@rakita rakita changed the base branch from main to rakita/count-insp-bench July 18, 2025 13:17
@codspeed-hq
Copy link

codspeed-hq bot commented Jul 18, 2025

CodSpeed Performance Report

Merging #2731 will not alter performance

Comparing rakita/insp-simple-end (7f0fa14) with main (f8a8f18)

Summary

✅ 171 untouched benchmarks

Base automatically changed from rakita/count-insp-bench to main July 18, 2025 13:36
@rakita
Copy link
Member Author

rakita commented Jul 28, 2025

Superseeded by #2797

@rakita rakita closed this Jul 28, 2025
@rakita rakita deleted the rakita/insp-simple-end branch July 30, 2025 19:21
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