Summary

Tests: Assert failure in library-test.scm on MinGW

Metadata

Attachments

Description

 
 "gcc" "a.c" -o "a.o" -c  -fno-strict-aliasing -fwrapv -DHAVE_CHICKEN_CONFIG_H -DC_ENABLE_PTABLES -march=native -O2 -IC:/Dev/MinGW32/include -IC:/Dev/GnuWin32/include -I.. -I"C:/Dev/Chicken/include/chicken"
 rm a.c
 "gcc" "a.o" -o "a.out" -Wl,--enable-auto-import -LC:/Dev/MinGW32/lib -LC:/Dev/GnuWin32/lib -L.. -L"C:/Dev/Chicken/lib" -lchicken -lm -lws2_32
 rm a.o
 
 Error: (library-tests.scm:204) assertion failed: (= (atan 42. 1.2) (fpatan2 42. 1.2))

Some naive testing of the above in csi and csc yields identical numbers from both functions. MinGW is lastest stable.

Changes and comments

[2012-07-02 17:02:23 UTC] Taverius attached check.log (description=Log of entire make check run.)

[2012-07-02 17:11:19 UTC] Taverius wrote:

Since I was unclear in description, this is core @ ac0a3d132f772da5c543288af6d0d33f66c8fecb from Sat, June 30.

[2012-07-09 16:30:35 UTC] Taverius attached a.zip (description=As requested, .c output attached)

[2012-07-09 19:30:21 UTC] sjamaan wrote:

This C output, when compiled with gcc on a BSD box and linked against master's libchicken runs and exits with exit code 0, without printing anything. This is the expected behavior, so the code generation is fine.

[2012-07-10 15:03:40 UTC] Taverius wrote:

I added

 printf("%lf, %lf\n",  C_flonum_magnitude(t3), C_flonum_magnitude(t4));

to the else branch as per your request, and manually compiled using the gcc calls from the log:

 c:\Dev\_src\chicken\trunk\tests>a.exe
 -0.000000, 0.000000

Something seems off here ...

[2012-07-10 21:27:01 UTC] sjamaan wrote:

Strangely, the value printed by (print (atan 42. 1.2)) is 1.5422326689561365 in both interpreted and compiled mode...

[2012-07-11 15:51:27 UTC] Taverius wrote:

Problem caused by -O2, close?

[2012-07-11 15:54:47 UTC] sjamaan wrote:

Replying to Taverius: > Problem caused by -O2, close?

Unsure. Felix?

[2012-07-11 16:28:12 UTC] sjamaan wrote:

Never mind, we just discussed this and it turns out the -O2 is due to local tweaks to the MingW system that cause it to build with -O2; Chicken doesn't do that by default.

[2012-07-11 16:28:12 UTC] sjamaan changed status from new to closed

[2012-07-11 16:28:12 UTC] sjamaan set resolution to invalid

[2012-09-24 21:47:48 UTC] felix changed milestone from 4.8.0 to 4.9.0

[2012-09-24 21:47:48 UTC] felix wrote:

Milestone 4.8.0 deleted