On 10.09.2012 11:14, Don Clugston wrote:
On 10 September 2012 10:33, GitHub <[email protected]> wrote:
   Log Message:
   -----------
   Fix issue 8624 Regression: CTFE: long comparisons completely broken

An implicit narrowing cast from long to int. Catastrophic.

We might need to do an emergency compiler release because of this bug.
It's one of the worst I've ever seen.


+        {   dinteger_t c = ee1->toInteger() - ee2->toInteger();
+            if (c > 0)
+                return 1;
+            if (c < 0)
+                return 1;

Should it return -1 in the second case? It seems the test suite doesn't do a lot of CTFE long compares...

_______________________________________________
dmd-internals mailing list
[email protected]
http://lists.puremagic.com/mailman/listinfo/dmd-internals

Reply via email to