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

gh-93678: move normalize_basic_block and extend_block call into optimize_cfg #95002

Merged
merged 6 commits into from
Jul 19, 2022

Conversation

iritkatriel
Copy link
Member

@iritkatriel iritkatriel commented Jul 19, 2022

This collects all cfg related code into a single function that can be exposed to python for unit testing.

Also updated a couple of obsolete comments and arranged all line number related calculations together.

@iritkatriel iritkatriel requested a review from markshannon as a code owner July 19, 2022 10:02
@iritkatriel iritkatriel added skip news interpreter-core (Objects, Python, Grammar, and Parser dirs) and removed awaiting core review labels Jul 19, 2022
@iritkatriel iritkatriel added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 19, 2022
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @iritkatriel for commit ba398ec 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 19, 2022
@markshannon markshannon self-requested a review July 19, 2022 16:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs) skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants