[Issue 4506] Regression(2.034): -O flag breaks some recursive functions.

d-bugmail at puremagic.com d-bugmail at puremagic.com
Sun Jul 25 15:29:58 PDT 2010


http://d.puremagic.com/issues/show_bug.cgi?id=4506


Don <clugdbug at yahoo.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch


--- Comment #4 from Don <clugdbug at yahoo.com.au> 2010-07-25 15:29:54 PDT ---
A one-line change in 2.034 which caused the regression. I think it's because
c *= -1; becomes NEGATE c, which isn't a binary operation. Here's a patch which
undoes the change. I'm not sure which bug it was attempting to fix.

gother.c, line 480, inside chkprop().

        if (d->Eoper == OPasm)          /* OPasm elems ruin everything  */
            goto noprop;
-        if (OTassign(d->Eoper) && EBIN(d))      // if assignment elem
+        if (OTassign(d->Eoper))      // if assignment elem
        {   elem *t = Elvalue(d);

            if (t->Eoper == OPvar)

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------


More information about the Digitalmars-d-bugs mailing list