FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Revert "gh-132775: Add _PyCode_GetVarCounts() (gh-133128)" by ericsnowcurrently · Pull Request #133232 · python/cpython · GitHub

/ cpython Public

Revert "gh-132775: Add _PyCode_GetVarCounts() (gh-133128)" - #133232

Merged
ericsnowcurrently merged 1 commit into
python:mainfrom
ericsnowcurrently:revert-add-pycode-get-var-counts
May 1, 2025
Merged

Revert "gh-132775: Add _PyCode_GetVarCounts() (gh-133128)"#133232
ericsnowcurrently merged 1 commit into
python:mainfrom
ericsnowcurrently:revert-add-pycode-get-var-counts

Conversation

Copy link
Copy Markdown
Member

The change broke the s390 builds, so I'm reverting it while I investigate.

ericsnowcurrently enabled auto-merge (squash) May 1, 2025 02:09
ericsnowcurrently merged commit 811edcf into python:main May 1, 2025
ericsnowcurrently deleted the revert-add-pycode-get-var-counts branch May 1, 2025 02:35
ericsnowcurrently added a commit that referenced this pull request May 5, 2025
This reverts commit 811edcf (gh-133232), which itself reverted the original commit 811edcf (gh-133128).

We reverted the original change due to failing s390 builds (a big-endian architecture).
It ended up that I had not accommodated op caches.
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
This reverts commit 811edcf (pythongh-133232), which itself reverted the original commit 811edcf (pythongh-133128).

We reverted the original change due to failing s390 builds (a big-endian architecture).
It ended up that I had not accommodated op caches.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL