Message82669
Okay, let's abandon 30-bit digits on 32-bit machines: it's still
unclear whether there's any real performance gain, and it's trivial to
re-enable 30-bit digits by default later. I'm also going to abandon the
optimizations for now; it'll be much easier to work on them once the base patch is in.
Here's a 'release-candidate' version of the patch: it's exactly the
same as before, except:
- I removed all x_divrem changes (though I've left the extra division
tests in, since they're just as relevant to the old x_divrem).
I'll open a separate issue for these optimizations.
- enable 30-bit digits by default only on 64-bit platforms, where
for the purposes of this patch a 64-bit platform is one with all
the necessary integer types available (signed and unsigned 32-
and 64-bit integer types) and SIZEOF_VOID_P >= 8.
I've also updated the version uploaded to Rietveld: the patchset there
should exactly match 30bit_longdigit20.patch. See:
http://codereview.appspot.com/14105
Martin, thank you for all your help with reviewing the previous patch.
Is it okay with you to check this version in? It's the same as the
version that you reviewed, except with some extra tests for correct
division, and with 30-bit digits disabled by default on 32-bit
platforms.
Gregory, if you have time, please could you double check that the
configure stuff is working okay for you with this patch? "configure --
enable-big-digits" should produce 30-bit digits; "configure --disable-
big-digits" should produce 15-bit digits, and a plain "configure" should
give 15-bit or 30-bit depending on your platform.
Anyone else have any objections to this going in exactly as it is? I'd
quite like to resolve this issue one way or the other and move on. |
|
Date |
User |
Action |
Args |
2009-02-24 18:19:44 | mark.dickinson | set | recipients:
+ mark.dickinson, tim.peters, loewis, collinwinter, gregory.p.smith, pitrou, pernici, vstinner, christian.heimes, jyasskin, schuppenies |
2009-02-24 18:19:44 | mark.dickinson | set | messageid: <1235499584.01.0.0921084318755.issue4258@psf.upfronthosting.co.za> |
2009-02-24 18:18:30 | mark.dickinson | link | issue4258 messages |
2009-02-24 18:18:27 | mark.dickinson | create | |
|