Title: Drop support for platforms without two's complement integer representation: require two's complement to build Python · Issue #100008 · python/cpython · GitHub
Open Graph Title: Drop support for platforms without two's complement integer representation: require two's complement to build Python · Issue #100008 · python/cpython
X Title: Drop support for platforms without two's complement integer representation: require two's complement to build Python · Issue #100008 · python/cpython
Description: Require Two's complement to build Python. Only very old machines (built in the 1960s?) like UNIVAC 1100/2200 series use signed number representations different than two's complement. Nowadays, all CPUs use the two's complement representa...
Open Graph Description: Require Two's complement to build Python. Only very old machines (built in the 1960s?) like UNIVAC 1100/2200 series use signed number representations different than two's complement. Nowadays, all ...
X Description: Require Two's complement to build Python. Only very old machines (built in the 1960s?) like UNIVAC 1100/2200 series use signed number representations different than two's complement. Nowada...
Opengraph URL: https://github.com/python/cpython/issues/100008
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Drop support for platforms without two's complement integer representation: require two's complement to build Python","articleBody":"Require [Two's complement](https://en.wikipedia.org/wiki/Two%27s_complement) to build Python.\r\n\r\nOnly very old machines (built in the 1960s?) like [UNIVAC 1100/2200 series](https://en.wikipedia.org/wiki/UNIVAC_1100/2200_series) use [signed number representations](https://en.wikipedia.org/wiki/Signed_number_representations) different than two's complement. Nowadays, all CPUs use the two's complement representation and CPython code base already relies on that in many places (especially ``Objects/longobject.c``). For example, Python adds the [-fwrapv compiler flag](https://gcc.gnu.org/onlinedocs/gcc/Code-Gen-Options.html) to GCC and clang.\r\n\r\nThe *signed* parameter of ``int.from_bytes()`` and ``int.to_bytes()`` indicates whether two's complement is used to represent the integer.\r\n\r\nI propose to be more explicit on build requirements for integers:\r\n\r\n* Two's complement\r\n* No trap representation\r\n* No padding bits\r\n\r\nMark Dickinson @mdickinson likes to repeat that CPython has these requirements :-)\r\n\r\n* https://github.com/python/cpython/pull/27832#pullrequestreview-734243048\r\n\r\n\u003e I think the `-(x + 1)` pattern came from days when we worried too much about alternative integer representations allowed by the C spec **(sign-magnitude, ones' complement, two's complement with a trap representation, etc.)**. But since then other parts of the codebase have been happy to assume that all integer types are **two's complement, no padding bits, no trap representation**, so I think it's safe to do so here. (And your changes to the other bitwise operations already assume two's complement and no trap representation.)\r\n\r\n* https://github.com/python/cpython/pull/17933#discussion_r674992923\r\n\r\n\u003e (...) But making all our usual assumptions about integer representation **(two's complement, no trap representation, no padding bits, etc.)**, `-INT_MIN-1` is the same as `INT_MAX` (...)\r\n\r\nI created this issue while reviewing PR #99762 which proposes to generalize a micro-optimization relying on a cast from an signed integer to an unsigned integer: replace ``0 \u003c= index \u0026\u0026 index \u003c limit`` (``Py_ssize_t``) with ``(size_t)index \u003c (size_t)limit``. I'm not sure that two's complement is strictly required for this micro-optimization, but it reminded it to me :-)\r\n\r\ncc @mdickinson\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-100009\n* gh-100014\n* gh-100045\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/vstinner","@type":"Person","name":"vstinner"},"datePublished":"2022-12-05T14:04:32.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":5},"url":"https://github.com/100008/cpython/issues/100008"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:fb37b003-077e-bde9-1938-d918db4f644e |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 9E9C:7F615:9805A9:CC8E66:696991ED |
| html-safe-nonce | 5f890891533c95a258b3b671d1ca114c6d2e719dfe8bcaffc51ead8301f89024 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5RTlDOjdGNjE1Ojk4MDVBOTpDQzhFNjY6Njk2OTkxRUQiLCJ2aXNpdG9yX2lkIjoiNDE4OTg0NDU1MDEwOTk5MTQwNSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | c800a0416800e56ad6668a2f9a1d80c204504453d381d196d8c1fbfc86d842d1 |
| hovercard-subject-tag | issue:1476646469 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/100008/issue_layout |
| twitter:image | https://opengraph.githubassets.com/2a99bab61353d4a9bfd5f8f5be0c5e547b186905a7b8b17a23ced155fbdb449c/python/cpython/issues/100008 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/2a99bab61353d4a9bfd5f8f5be0c5e547b186905a7b8b17a23ced155fbdb449c/python/cpython/issues/100008 |
| og:image:alt | Require Two's complement to build Python. Only very old machines (built in the 1960s?) like UNIVAC 1100/2200 series use signed number representations different than two's complement. Nowadays, all ... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | vstinner |
| hostname | github.com |
| expected-hostname | github.com |
| None | 3542e147982176a7ebaa23dfb559c8af16f721c03ec560c68c56b64a0f35e751 |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | af80af7cc9e3de9c336f18b208a600950a3c187c |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width