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

bpo-44313: generate LOAD_ATTR/CALL_FUNCTION for top-level imported objects #26677

Merged
merged 2 commits into from Jun 30, 2021

Conversation

isidentical
Copy link
Sponsor Member

@isidentical isidentical commented Jun 11, 2021

@isidentical isidentical requested a review from markshannon Jun 11, 2021
@isidentical isidentical changed the title bpo-44313: generate LOAD_ATTR/CALL_FUNCTION for top-level imported ob… bpo-44313: generate LOAD_ATTR/CALL_FUNCTION for top-level imported objects Jun 11, 2021
@isidentical isidentical force-pushed the bpo-44313 branch 4 times, most recently from 86f2a4d to c7c708c Compare Jun 11, 2021
Copy link
Member

@markshannon markshannon left a comment

Great. Just two very minor issues.

Lib/test/test_compile.py Outdated Show resolved Hide resolved
@bedevere-bot
Copy link

bedevere-bot commented Jun 14, 2021

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@isidentical
Copy link
Sponsor Member Author

isidentical commented Jun 18, 2021

I have made the requested changes; please review again

@bedevere-bot
Copy link

bedevere-bot commented Jun 18, 2021

Thanks for making the requested changes!

@markshannon: please review the changes made to this pull request.

Copy link
Member

@markshannon markshannon left a comment

Looks good now.
Sorry I missed your update, so importlib needs updating again.

@isidentical
Copy link
Sponsor Member Author

isidentical commented Jun 29, 2021

Okay will regenerate them tomorrow.

@isidentical isidentical added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jun 30, 2021
@bedevere-bot
Copy link

bedevere-bot commented Jun 30, 2021

🤖 New build scheduled with the buildbot fleet by @isidentical for commit ab1b6c1 🤖

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

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jun 30, 2021
@markshannon
Copy link
Member

markshannon commented Jun 30, 2021

Thanks.

@markshannon markshannon merged commit 1b28187 into python:main Jun 30, 2021
73 of 74 checks passed
@@ -360,6 +360,7 @@ def _write_atomic(path, data, mode=0o666):
# Python 3.11a1 3455 (add MAKE_CELL bpo-43693)
# Python 3.11a1 3456 (interleave cell args bpo-43693)
# Python 3.11a1 3457 (Change localsplus to a bytes object bpo-43693)
# Python 3.11a1 3458 (imported objects now don't use LOAD_METHOD/CALL_METHOD)
Copy link
Member

@sweeneyde sweeneyde Jul 1, 2021

Choose a reason for hiding this comment

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

Shouldn't the actual number be bumped below? I don't think it matters much in this case, but for the sake of consistency/clarity for the next bump, it might be good to fix.

Copy link
Sponsor Member Author

@isidentical isidentical Jul 1, 2021

Choose a reason for hiding this comment

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

Right, probably got away during rebase. Will fix

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.

None yet

6 participants