Skip to content

Conversation

@iritkatriel
Copy link
Member

@iritkatrieliritkatriel commented Sep 3, 2024

This is The Big Split.

In order to preserve as much git history as possible, I did a git mv Python/compile.c Python/codegen.c. Then I copied the compiler code from codegen.c to compile.c.

We end up with:

% wc -l Python/codegen.c Python/compile.c 6142 Python/codegen.c 1665 Python/compile.c 

So a complex file of 1.6K lines, and a relatively simple file with 6K lines.

@JelleZijlstra
Copy link
Member

FYI there are some compile errors (I'll do an actual review later)

@iritkatrieliritkatriel added skip news interpreter-core (Objects, Python, Grammar, and Parser dirs) labels Sep 3, 2024
Copy link
Member

@picnixzpicnixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you check if the NO_LABEL entry in ignored.tsv for compile.c should be removed or not?

Copy link
Member

@JelleZijlstraJelleZijlstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, will leave remaining decisions on prefixes up to you.

@iritkatrieliritkatriel merged commit 1a9d891 into python:mainSep 9, 2024
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Sep 10, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in 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

@iritkatriel@JelleZijlstra@picnixz