ArrayFunctionsTest::testDubiousArrayFormulasFODS fails on ppc64le (and s390x)
Stephan Bergmann
sbergman at redhat.com
Fri Sep 1 14:36:30 UTC 2023
On 9/1/23 15:41, Dan Horák wrote:
> On Fri, 1 Sep 2023 11:29:23 +0200
> Stephan Bergmann <sbergman at redhat.com> wrote:
[...]
>> would be that the TODO comment makes it then sound like the issues on
>> ppc64le and s390x are also known to be due to -ffp-contract settings,
>> even if you would not verify that.
>
> I have verified it's indeed caused by -ffp-contract. When set to "off",
> the test passes (checked on ppc64le).
Good to know. Please add that knowledge to the commit message.
>> So maybe the best thing is to turn this into a x86_64 only conditional,
>> and extend the TODO comment with something like "...on at least aarch64,
>> so lets only execute this on x86-64, where it appears to always work well:"
>
> yeah, I think using a "positive" list would be better. I could
> possibly prepare a patch, but is there an example where I can see how
> the condition should look like?
For better or worse, we pass or configure.ac's "compiler independent"
CPUNAME into the compiler via -D$(CPUNAME) (solenv/gbuild/gbuild.mk), so
something like
> #if defined X86_64
should work for a positive list of just x86_64, regardless of compiler
(MSVC vs. Clang/GCC).
More information about the LibreOffice
mailing list