diff options
author | Tim Peters <tim.peters@gmail.com> | 2000-07-08 04:17:21 (GMT) |
---|---|---|
committer | Tim Peters <tim.peters@gmail.com> | 2000-07-08 04:17:21 (GMT) |
commit | 7d3a511a40a8c90eac66d3d59edbbe3c3d4559b1 (patch) | |
tree | b2a404e6604682d51adf242e1ac51ac225e6a0bc /Objects/intobject.c | |
parent | 5639ba4896fa7a4e29f11bf0c42f6e3125785654 (diff) | |
download | cpython-7d3a511a40a8c90eac66d3d59edbbe3c3d4559b1.zip cpython-7d3a511a40a8c90eac66d3d59edbbe3c3d4559b1.tar.gz cpython-7d3a511a40a8c90eac66d3d59edbbe3c3d4559b1.tar.bz2 |
Cray J90 fixes for long ints.
This was a convenient excuse to create the pyport.h file recently
discussed!
Please use new Py_ARITHMETIC_RIGHT_SHIFT when right-shifting a
signed int and you *need* sign-extension. This is #define'd in
pyport.h, keying off new config symbol SIGNED_RIGHT_SHIFT_ZERO_FILLS.
If you're running on a platform that needs that symbol #define'd,
the std tests never would have worked for you (in particular,
at least test_long would have failed).
The autoconfig stuff got added to Python after my Unix days, so
I don't know how that works. Would someone please look into doing
& testing an auto-config of the SIGNED_RIGHT_SHIFT_ZERO_FILLS
symbol? It needs to be defined if & only if, e.g., (-1) >> 3 is
not -1.
Diffstat (limited to 'Objects/intobject.c')
-rw-r--r-- | Objects/intobject.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/Objects/intobject.c b/Objects/intobject.c index ba3dde1..a5dc134 100644 --- a/Objects/intobject.c +++ b/Objects/intobject.c @@ -721,10 +721,7 @@ int_rshift(v, w) a = 0; } else { - if (a < 0) - a = ~( ~(unsigned long)a >> b ); - else - a = (unsigned long)a >> b; + a = Py_ARITHMETIC_RIGHT_SHIFT(long, a, b); } return PyInt_FromLong(a); } |