diff options
author | Eike Rathke <erack@redhat.com> | 2023-10-15 17:29:01 +0200 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2023-10-15 18:46:01 +0200 |
commit | 361c4f008e48b08df635839d2e5dcad7389df44a (patch) | |
tree | 8e416ceac9b8b2c912ee11c4fcf9214f2061d25d /compilerplugins/clang/check.cxx | |
parent | 59480a5e4ebb849c06d7f16579208d0847c5557e (diff) |
Follow-up: tdf#156985 Use SC_USE_SSE2 to determine which KahanSum::add() to use
Also, the CPU identifier for MSVC WIN32 is not X86 but INTEL, so
actually use SSE2 there as well, which was the cause of things
failing on that platform.
For other platforms than Intel x86/x86_64 SSE2 is not defined, so
exclude the new unit test based on that and live on with the old
slightly off value. Experiments did not yield any solution that
works, even using plain sumNeumaierNormal() (similar to SSE2) in
the executeUnrolled() case instead of KahanSum with its m_fMem did
not help, nor trying to add the internal values in different
orders or with long double, au contraire the error was slightly
larger.
Change-Id: Ica0b2963f76c01f248799e9a809ef06eb099e722
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156899
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
Diffstat (limited to 'compilerplugins/clang/check.cxx')
0 files changed, 0 insertions, 0 deletions