diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2020-05-05 22:24:50 +0200 |
---|---|---|
committer | Robert C. Helling <helling@atdotde.de> | 2020-05-06 00:22:31 +0200 |
commit | 123937901fe9c35fcb1be471fb6ad8a9f89f1b1d (patch) | |
tree | 475940d30ac8830063e9053d52b308dfd6cf48b6 /.clang-format | |
parent | f37f42876260e804e4e990d608a8dcd21a141ba0 (diff) | |
download | subsurface-123937901fe9c35fcb1be471fb6ad8a9f89f1b1d.tar.gz |
planner: remove Bühlmann factor cache
The Bühlmann factors were cached in a thread-safe hashmap. It seemed
somewhat dubious that entering a critical section and doing a hash-lookup
would be significantly faster than a simple exp() call.
Indeed, in a very cache friendly test (16 entries, tight loop) calling the
factor() function 32 000 000 times from a different translation units we get:
- with cache: 604 ms
- without cache: 266 ms
Therefore, remove the cache. Given that 32 000 000 calls take only 266 ms,
it appears not sensible to try to optimize this function anyway.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to '.clang-format')
0 files changed, 0 insertions, 0 deletions