IntegerTest>>#testExactSqrt broken on 32bit trunk

Hi all,

It looks like there's a large integer-related problem in 32bit trunk (see screenshot and [1]). I'm assuming this was introduced with the recent changes Nicolas incorporated. Maybe someone could have a look? For reproducing, download the last 32bit trunk image [2], update it, and run IntegerTest>>#testExactSqrt.

Since 64bit works, we finally have a new 64bit trunk build (see [3]).

Cheers,

Fabio

[1] https://travis-ci.org/squeak-smalltalk/squeak-app/jobs/536301595#L2512

[2] http://files.squeak.org/5.3alpha/Squeak5.3alpha-18498-32bit/

[3] http://files.squeak.org/5.3alpha/Squeak5.3alpha-18592-64bit/

Hi Fabio,

It seems to just be a timeOut.

I'll have a look.

Le jeu. 23 mai 2019 à 17:05, Fabio Niephaus <[hidden email]> a écrit :

Hi all,

It looks like there's a large integer-related problem in 32bit trunk (see screenshot and [1]). I'm assuming this was introduced with the recent changes Nicolas incorporated. Maybe someone could have a look? For reproducing, download the last 32bit trunk image [2], update it, and run IntegerTest>>#testExactSqrt.

Since 64bit works, we finally have a new 64bit trunk build (see [3]).

Cheers,

Fabio

[1] https://travis-ci.org/squeak-smalltalk/squeak-app/jobs/536301595#L2512

[2] http://files.squeak.org/5.3alpha/Squeak5.3alpha-18498-32bit/

[3] http://files.squeak.org/5.3alpha/Squeak5.3alpha-18592-64bit/

Ah, it's an infinite loop on 32bits, I'll commit a patch soon.

Le jeu. 23 mai 2019 à 18:40, Nicolas Cellier <[hidden email]> a écrit :

Hi Fabio,

It seems to just be a timeOut.

I'll have a look.

Le jeu. 23 mai 2019 à 17:05, Fabio Niephaus <[hidden email]> a écrit :

Hi all,

It looks like there's a large integer-related problem in 32bit trunk (see screenshot and [1]). I'm assuming this was introduced with the recent changes Nicolas incorporated. Maybe someone could have a look? For reproducing, download the last 32bit trunk image [2], update it, and run IntegerTest>>#testExactSqrt.

Since 64bit works, we finally have a new 64bit trunk build (see [3]).

Cheers,

Fabio

[1] https://travis-ci.org/squeak-smalltalk/squeak-app/jobs/536301595#L2512

[2] http://files.squeak.org/5.3alpha/Squeak5.3alpha-18498-32bit/

[3] http://files.squeak.org/5.3alpha/Squeak5.3alpha-18592-64bit/