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


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

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


--- Comment #9 from Don <clugdbug@yahoo.com.au> 2011-04-08 21:22:35 PDT ---
(In reply to comment #8)
> Don's still-in-progress CTFE overhaul seems to fix this: https://github.com/donc/dmd/tree/ctfemem

Yes, it fixes almost all CTFE bugs involving arrays.

Fixed.
D1:
https://github.com/D-Programming-Language/dmd/commit/ef2a0f1da10331c4de102ca3e029ea1c1c1a46bf

D2: https://github.com/D-Programming-Language/dmd/commit/0219a5f0dcc88076759a4c472582954d0fb804b0

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

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


Jimmy Cao <jcao219@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jcao219@gmail.com


--- Comment #10 from Jimmy Cao <jcao219@gmail.com> 2011-05-29 19:02:16 PDT ---
It doesn't seem like this is fixed in dmd 2.053, but it should be according to the changelog.

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

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



--- Comment #11 from Don <clugdbug@yahoo.com.au> 2011-05-29 21:19:24 PDT ---
(In reply to comment #10)
> It doesn't seem like this is fixed in dmd 2.053, but it should be according to the changelog.

Can you describe what's failing? The test in comment 7 works for me.

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

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



--- Comment #12 from Jimmy Cao <jcao219@gmail.com> 2011-05-29 21:31:41 PDT ---
The test in comment 7 asserts for me.
I'm using dmd 2.053 on Windows.

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

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



--- Comment #13 from Jimmy Cao <jcao219@gmail.com> 2011-05-29 22:32:24 PDT ---
It doesn't work on Fedora 15 either.

I'm using dmd 2.053 directly from the zip file on the digitalmars site (for both Fedora 15 and Windows).

Don, is your dmd built from source?  Maybe there's a difference between mine and yours.

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

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


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

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


--- Comment #14 from Don <clugdbug@yahoo.com.au> 2011-05-30 07:54:25 PDT ---
I think there was something wrong with that test case, it wasn't instantiating the template. Here's a reduced test case:

string b(string s) { return s; }
string a(string s) { return b(s); }

void bug5708() {
    void m() { a("lit"); }
    static assert(a("foo") == "foo");
    static assert(a("bar") == "bar");
}

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

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


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED
            Summary|Incorrect string constant   |[CTFE] Incorrect string
                   |folding with -inline        |constant folding with
                   |                            |-inline


--- Comment #15 from Don <clugdbug@yahoo.com.au> 2011-06-26 14:56:40 PDT ---
It was accidentally fixed in this commit:
6072 - [CTFE] Regression(git master): Cannot declare variable inside an 'if'
condition

which actually fixed the treatment of comma expressions in CTFE.

https://github.com/D-Programming-Language/dmd/commit/1bcae90e09f5eb6988fcd6df4309a137c9901084

I'm adding this to the test suite so it stays fixed.

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