[lmi-commits] [lmi] master e0ea157 7/7: Switch to currency class

[Greg Chicares]

branch: master
commit e0ea157c53af9149efdeac123fb9e1e0b542fef9
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Switch to currency class
    
    Compared to master dfdbf3c5a, performance is slower by about:
    
                                  no    some    many
                                solve  solves  solves
    gcc_i686-w64-mingw32          6%     11%     16%
    gcc_x86_64-pc-linux-gnu      11%     23%     32%
    gcc_x86_64-w64-mingw32        8%     19%     28%
    
    Search online for "Raymond's last day as the band's sound technician":
    that's what this may look like at first blush. However:
     - "First make it right, then make it fast": not much effort has yet
       been spent on making this fast.
     - The valyuta branches contain some important speed improvements that
       will soon be brought over.
     - Replacing i686 with x86_64-w64-mingw32 in production will double the
       speed anyway.
---
 Speed_gcc_i686-w64-mingw32    | 12 ++++++------
 Speed_gcc_x86_64-pc-linux-gnu | 12 ++++++------
 Speed_gcc_x86_64-w64-mingw32  | 12 ++++++------
 currency.hpp                  |  2 +-
 4 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/Speed_gcc_i686-w64-mingw32 b/Speed_gcc_i686-w64-mingw32
index 7852abd..ec20225 100644
--- a/Speed_gcc_i686-w64-mingw32
+++ b/Speed_gcc_i686-w64-mingw32
@@ -1,7 +1,7 @@
 Test speed:
-  naic, no solve      : 5.760e-02 s mean;      56739 us least of  18 runs
-  naic, specamt solve : 1.019e-01 s mean;     100864 us least of  10 runs
-  naic, ee prem solve : 9.356e-02 s mean;      92648 us least of  11 runs
-  finra, no solve     : 2.333e-02 s mean;      22284 us least of  44 runs
-  finra, specamt solve: 6.338e-02 s mean;      62696 us least of  16 runs
-  finra, ee prem solve: 5.888e-02 s mean;      58346 us least of  17 runs
+  naic, no solve      : 6.140e-02 s mean;      60962 us least of  17 runs
+  naic, specamt solve : 1.087e-01 s mean;     107801 us least of  10 runs
+  naic, ee prem solve : 9.953e-02 s mean;      98685 us least of  11 runs
+  finra, no solve     : 2.308e-02 s mean;      22885 us least of  44 runs
+  finra, specamt solve: 6.607e-02 s mean;      65418 us least of  16 runs
+  finra, ee prem solve: 6.131e-02 s mean;      60856 us least of  17 runs
diff --git a/Speed_gcc_x86_64-pc-linux-gnu b/Speed_gcc_x86_64-pc-linux-gnu
index 86ca190..7255cdc 100644
--- a/Speed_gcc_x86_64-pc-linux-gnu
+++ b/Speed_gcc_x86_64-pc-linux-gnu
@@ -1,7 +1,7 @@
 Test speed:
-  naic, no solve      : 2.300e-02 s mean;      21597 us least of  44 runs
-  naic, specamt solve : 3.815e-02 s mean;      37672 us least of  27 runs
-  naic, ee prem solve : 3.493e-02 s mean;      34426 us least of  29 runs
-  finra, no solve     : 7.997e-03 s mean;       7544 us least of 100 runs
-  finra, specamt solve: 2.273e-02 s mean;      22038 us least of  44 runs
-  finra, ee prem solve: 2.131e-02 s mean;      20505 us least of  47 runs
+  naic, no solve      : 2.519e-02 s mean;      24940 us least of  40 runs
+  naic, specamt solve : 4.382e-02 s mean;      43197 us least of  23 runs
+  naic, ee prem solve : 4.004e-02 s mean;      39716 us least of  25 runs
+  finra, no solve     : 8.380e-03 s mean;       8012 us least of 100 runs
+  finra, specamt solve: 2.515e-02 s mean;      24774 us least of  40 runs
+  finra, ee prem solve: 2.347e-02 s mean;      22908 us least of  43 runs
diff --git a/Speed_gcc_x86_64-w64-mingw32 b/Speed_gcc_x86_64-w64-mingw32
index a44a4e7..172e656 100644
--- a/Speed_gcc_x86_64-w64-mingw32
+++ b/Speed_gcc_x86_64-w64-mingw32
@@ -1,7 +1,7 @@
 Test speed:
-  naic, no solve      : 3.188e-02 s mean;      31652 us least of  32 runs
-  naic, specamt solve : 5.079e-02 s mean;      50612 us least of  20 runs
-  naic, ee prem solve : 4.711e-02 s mean;      46856 us least of  22 runs
-  finra, no solve     : 1.549e-02 s mean;      15272 us least of  65 runs
-  finra, specamt solve: 3.264e-02 s mean;      32352 us least of  31 runs
-  finra, ee prem solve: 3.077e-02 s mean;      30530 us least of  33 runs
+  naic, no solve      : 3.386e-02 s mean;      33629 us least of  30 runs
+  naic, specamt solve : 5.437e-02 s mean;      54066 us least of  19 runs
+  naic, ee prem solve : 5.036e-02 s mean;      50030 us least of  20 runs
+  finra, no solve     : 1.630e-02 s mean;      15494 us least of  62 runs
+  finra, specamt solve: 3.428e-02 s mean;      34090 us least of  30 runs
+  finra, ee prem solve: 3.225e-02 s mean;      32052 us least of  32 runs
diff --git a/currency.hpp b/currency.hpp
index 1183868..e3d45df 100644
--- a/currency.hpp
+++ b/currency.hpp
@@ -33,7 +33,7 @@
 // elsewhere. Eventually they'll both be eliminated, along with
 // all code along paths where they aren't both defined.
 
-// #define USE_CURRENCY_CLASS
+#define USE_CURRENCY_CLASS
 
 #if !defined USE_CURRENCY_CLASS
 #   undef CURRENCY_UNIT_IS_CENTS // Requires currency class.