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

gh-108562: Revert enabling -fstrict-overflow for libmpdec by skirpichev · Pull Request #116302 · python/cpython · GitHub

/ cpython Public

gh-108562: Revert enabling -fstrict-overflow for libmpdec - #116302

Merged
encukou merged 1 commit into
python:mainfrom
skirpichev:partial-revert-gh-108562
Mar 4, 2024
Merged

gh-108562: Revert enabling -fstrict-overflow for libmpdec#116302
encukou merged 1 commit into
python:mainfrom
skirpichev:partial-revert-gh-108562

Conversation

skirpichev commented Mar 4, 2024
edited by erlend-aasland
Loading

Copy link
Copy Markdown
Member

Reverts -fstrict-overflow for libmpdec, added in #114751.

Reverts -fstrict-overflow for libmpdec
erlend-aasland requested a review from encukou March 4, 2024 11:07
erlend-aasland changed the title gh-108562: partial reversion of pr114751 gh-108562: Revert enabling -fstrict-overflow for libmpdec #114751 Mar 4, 2024
erlend-aasland changed the title gh-108562: Revert enabling -fstrict-overflow for libmpdec #114751 gh-108562: Revert enabling -fstrict-overflow for libmpdec Mar 4, 2024

This comment was marked as outdated.

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @erlend-aasland for commit ca35e50 🤖

The command will test the builders whose names match following regular expression: .*LTO.*

The builders matched are:

  • s390x Fedora Rawhide LTO + PGO PR
  • PPC64LE RHEL8 LTO + PGO PR
  • PPC64LE Fedora Rawhide LTO + PGO PR
  • aarch64 Fedora Rawhide LTO PR
  • PPC64LE Fedora Stable LTO PR
  • aarch64 RHEL8 LTO + PGO PR
  • PPC64LE Fedora Rawhide LTO PR
  • s390x RHEL7 LTO PR
  • s390x Fedora LTO + PGO PR
  • AMD64 Fedora Rawhide LTO + PGO PR
  • AMD64 RHEL8 LTO + PGO PR
  • s390x RHEL8 LTO + PGO PR
  • AMD64 CentOS9 LTO + PGO PR
  • s390x Fedora Rawhide LTO PR
  • AMD64 Fedora Rawhide LTO PR
  • AMD64 RHEL7 LTO + PGO PR
  • AMD64 Fedora Stable LTO PR
  • AMD64 RHEL7 LTO PR
  • PPC64LE RHEL7 LTO + PGO PR
  • PPC64LE RHEL8 LTO PR
  • PPC64LE Fedora Stable LTO + PGO PR
  • aarch64 Debian Clang LTO + PGO PR
  • aarch64 Fedora Rawhide LTO + PGO PR
  • aarch64 CentOS9 LTO PR
  • s390x RHEL7 LTO + PGO PR
  • aarch64 RHEL8 LTO PR
  • PPC64LE RHEL7 LTO PR
  • AMD64 CentOS9 LTO PR
  • aarch64 Fedora Stable LTO PR
  • AMD64 Fedora Stable LTO + PGO PR
  • aarch64 CentOS9 LTO + PGO PR
  • PPC64LE CentOS9 LTO PR
  • AMD64 RHEL8 LTO PR
  • aarch64 Fedora Stable LTO + PGO PR
  • s390x RHEL8 LTO PR
  • PPC64LE CentOS9 LTO + PGO PR
  • s390x Fedora LTO PR

encukou commented Mar 4, 2024

Copy link
Copy Markdown
Member

(The posix.sched_setaffinity(0, []) on CentOS9 is unrelated, not a CPython issue.)

Copy link
Copy Markdown
Member

Is this ready to be merged?

encukou merged commit ea1b1c5 into python:main Mar 4, 2024

encukou commented Mar 4, 2024
edited
Loading

Copy link
Copy Markdown
Member

Yes. Thank you @skirpichev for the quick fix!

woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
…onGH-116302)

pythongh-108562: partial reversion of pr114751

Reverts -fstrict-overflow for libmpdec
skirpichev deleted the partial-revert-gh-108562 branch March 5, 2024 01:46

Copy link
Copy Markdown
Member Author

Thanks. Sorry for a breakage. Next time I will suggest running more extended tests.

adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
…onGH-116302)

pythongh-108562: partial reversion of pr114751

Reverts -fstrict-overflow for libmpdec
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…onGH-116302)

pythongh-108562: partial reversion of pr114751

Reverts -fstrict-overflow for libmpdec
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.

5 participants


Back | FazBrowse Home | New Git URL