http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #1 from Iain Buclaw <ibuclaw@gdcproject.org> 2013-04-04 14:50:18 UTC ---
Fixed GC-related problem in the compiler.

https://github.com/D-Programming-GDC/GDC/commit/0adf38c953db12a50e7fac48db321a49739f9b99


I wonder if those segv's you've found are now fixed.

Regards
Iain

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #2 from Johannes Pfau <johannespfau@gmail.com> 2013-04-04 18:45:59 UTC ---
I'll see if I can run the unit tests again but I used a custom gdc branch with patches which allows skipping failing tests. That code is quite outdated now so I have to update it first.

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #3 from Iain Buclaw <ibuclaw@gdcproject.org> 2013-04-05 11:10:37 UTC ---
Another GC-related fix, and turned on 'make check-target-libphobos' which should run the library testsuite.


https://github.com/D-Programming-GDC/GDC/commit/67ef85a98938fc1584a8e1f4dbd2b354a736eaf2

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #4 from Johannes Pfau <johannespfau@gmail.com> 2013-04-06 16:42:06 UTC ---
2 are fixed but I've not verified if they were fixed by the GC changes. I'll try to reduce the other two segvs and file new bug reports for them.

Overall the situation is much better now with only 13 failing tests:
std.complex:663               FAILED
std.complex:683               FAILED
std.typecons:3281             FAILED
std.range:2953                FAILED
std.range:8640                FAILED
std.math:327                  FAILED
std.math:441                  FAILED
std.math:1517                 FAILED
std.math:1528                 FAILED
std.math:1632                 FAILED
std.math:2999                 FAILED
std.conv:2487                 FAILED
std.conv:2622                 FAILED
Total results: Passed 1892 Failed 13 Disabled 2

./unittest --show-disabled
std.container:3623
std.algorithm:1721

Here's my fork with the "enhanced" unit test support in case you want to try
it:
https://github.com/jpf91/GDC/commits/newUnittest

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #5 from Iain Buclaw <ibuclaw@gdcproject.org> 2013-04-06 17:27:27 UTC ---
(In reply to comment #4)
> 2 are fixed but I've not verified if they were fixed by the GC changes. I'll try to reduce the other two segvs and file new bug reports for them.
> 
> Overall the situation is much better now with only 13 failing tests:
> std.complex:663               FAILED
> std.complex:683               FAILED
> std.typecons:3281             FAILED
> std.range:2953                FAILED
> std.range:8640                FAILED
> std.math:327                  FAILED
> std.math:441                  FAILED
> std.math:1517                 FAILED
> std.math:1528                 FAILED
> std.math:1632                 FAILED
> std.math:2999                 FAILED
> std.conv:2487                 FAILED
> std.conv:2622                 FAILED
> Total results: Passed 1892 Failed 13 Disabled 2
> 
> ./unittest --show-disabled
> std.container:3623
> std.algorithm:1721
> 
> Here's my fork with the "enhanced" unit test support in case you want to try
> it:
> https://github.com/jpf91/GDC/commits/newUnittest

Ah, but do they cause gdc to SEGV, or do they SEGV during runtime?

The fixes I did addressed SEGV during compilation of those two files you've shown to be disabled.

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #6 from Johannes Pfau <johannespfau@gmail.com> 2013-04-06 17:49:45 UTC ---
At runtime. There are no compiler ICEs or compiler segfaults when compiling the unittests AFAICS.

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #7 from Iain Buclaw <ibuclaw@gdcproject.org> 2013-04-06 18:20:54 UTC ---
(In reply to comment #6)
> At runtime. There are no compiler ICEs or compiler segfaults when compiling the unittests AFAICS.

Excellent.  That should now allow us to do proper testing now then.

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

--- Comment #8 from Iain Buclaw <ibuclaw@gdcproject.org> 2013-05-14 23:11:14 UTC ---
Updated results:

std/internal/math/errorfunction.d:222
FAILED
std/internal/math/gammafunction.d:367
FAILED
std/typecons.d:3293
FAILED
std/range.d:2995
FAILED
std/math.d:336
FAILED
std/conv.d:2524
FAILED

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.

http://bugzilla.gdcproject.org/show_bug.cgi?id=32

Johannes Pfau <johannespfau@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

-- 
Configure bugmail: http://bugzilla.gdcproject.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.