X Tutup
Skip to content

gh-108113: Make it possible to create an optimized AST#108154

Merged
iritkatriel merged 10 commits intopython:mainfrom
iritkatriel:ast_opt
Aug 21, 2023
Merged

gh-108113: Make it possible to create an optimized AST#108154
iritkatriel merged 10 commits intopython:mainfrom
iritkatriel:ast_opt

Conversation

@iritkatriel
Copy link
Copy Markdown
Member

@iritkatriel iritkatriel commented Aug 19, 2023

iritkatriel and others added 2 commits August 21, 2023 10:06
Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
Copy link
Copy Markdown
Member

@carljm carljm left a comment

Choose a reason for hiding this comment

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

Looks like there's a docs build issue, but otherwise this LGTM.

@iritkatriel iritkatriel enabled auto-merge (squash) August 21, 2023 15:49
@dreamflow
Copy link
Copy Markdown
Contributor

should the new ast.PyCF_OPTIMIZED_AST
be mentioned in the compiler-flags doc ?

the compile() doc has a link to the compiler-flags doc ,
so that someone reading the compile() doc , can find out
that compile() has the ast.PyCF_OPTIMIZED_AST option .

@iritkatriel
Copy link
Copy Markdown
Member Author

@dreamflow Would you like to create a PR for that doc change?

@dreamflow
Copy link
Copy Markdown
Contributor

@iritkatriel yes , i'm happy to 🙂

have posted the PR #113241

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.

compile() with PyCF_ONLY_AST flag ignores the optimize arg

5 participants

X Tutup