Re: recent KahanSum change causes a new test failure on ppc64le

[Dan Horák <dan@xxxxxxxx>]

On Wed, 6 Sep 2023 16:32:49 +0200
Xisco Fauli <xiscofauli@xxxxxxxxxxxxxxx> wrote:

> Hello Dan,
> 
> That test is indeed testing the patch you are reverting, so it's 
> expected to fail. It shouldn't fail if you also revert 
> 2ac7d339a97efa638124c2862551e0876487833b.

thanks, it makes sense (and helps) :-)


		Dan
 
> On 6/9/23 15:41, Dan Horák wrote:
> > On Tue, 5 Sep 2023 18:53:37 +0200
> > Dan Horák <dan@xxxxxxxx> wrote:
> >  
> >> Hello,
> >>
> >> seems the recent change [1] to KahanSum handling causes a test failure
> >> on ppc64le.
> >>
> >> [1] https://git.libreoffice.org/core/+/1f8cc7644293e62ad6430bbeec243d3283e478d7  
> > and with the change reverted I am getting a failure in
> > CppunitTest_sc_ucalc_formula2
> >
> > ...
> > testTdf147398::TestBody finished in: 11ms
> > [_RUN_____] testTdf156985::TestBody
> > /home/sharkcz/projects/libreoffice/sc/qa/unit/ucalc_formula2.cxx:
> > 4624:testTdf156985::TestBody equality assertion failed
> > - Expected: 0
> > - Actual  : -1.59872115546023e-14
> >
> > testTdf156985::TestBody finished in: 11ms
> > ...
> >
> > so not good ...
> >
> >
> > 		Dan  
> 
> -- 
> Xisco Faulí
> LibreOffice QA Team
> IRC: x1sc0
>