Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#879 closed defect (invalid)

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

Reported by: Leonardo Valeri Manera Owned by:
Priority: minor Milestone: 4.9.0
Component: unknown Version: 4.7.x
Keywords: Cc:
Estimated difficulty:

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.

Attachments (2)

check.log (18.3 KB ) - added by Leonardo Valeri Manera 13 years ago.
Log of entire make check run.
a.zip (67.3 KB ) - added by Leonardo Valeri Manera 13 years ago.
As requested, .c output attached

Download all attachments as: .zip

Change History (10)

by Leonardo Valeri Manera, 13 years ago

Attachment: check.log added

Log of entire make check run.

comment:1 by Leonardo Valeri Manera, 13 years ago

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

by Leonardo Valeri Manera, 13 years ago

Attachment: a.zip added

As requested, .c output attached

comment:2 by sjamaan, 13 years ago

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.

comment:3 by Leonardo Valeri Manera, 13 years ago

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 ...

comment:4 by sjamaan, 13 years ago

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

comment:5 by Leonardo Valeri Manera, 13 years ago

Problem caused by -O2, close?

in reply to:  5 comment:6 by sjamaan, 13 years ago

Replying to Taverius:

Problem caused by -O2, close?

Unsure. Felix?

comment:7 by sjamaan, 13 years ago

Resolution: invalid
Status: newclosed

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.

comment:8 by felix winkelmann, 13 years ago

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.