View mode: basic / threaded / horizontal-split · Log in · Help
November 20, 2012
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #10 from github-bugzilla@puremagic.com 2012-11-20 09:55:06 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/28dedee456e741f02f08a944f41daa9e46236224
Issue 7444 - Require [] for array copies too

https://github.com/D-Programming-Language/phobos/commit/9a6dad8a2ac5841bdcfa2b86082450818f6eefab
Merge pull request #960 from 9rnsr/fix7444

Supplemental changes for Issue 7444 - Require [] for array copies too

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
November 22, 2012
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #11 from github-bugzilla@puremagic.com 2012-11-21 20:12:48 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/druntime

https://github.com/D-Programming-Language/druntime/commit/e5895640c83d8b5b4c8b2404b4e639ba6fdf2243
Additional fix Issue 7444 in Posix platforms

https://github.com/D-Programming-Language/druntime/commit/a54f41c98aa26eeb70274e8b78d8abfb575bcebc
Merge pull request #352 from 9rnsr/fix7444

Additional fix Issue 7444 in Posix platforms

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 07, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #12 from github-bugzilla@puremagic.com 2013-03-06 22:53:51 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/57b770ed49379c5af726d23356e0f75818a3f859
Issue 7444 - Require [] for array copies too

https://github.com/D-Programming-Language/dmd/commit/ba1009c5561b51b8f18d9c869fde9bd45cb7ebc7
Merge pull request #702 from 9rnsr/fix7444

Issue 7444 - Require [] for array copies too

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 07, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #13 from bearophile_hugs@eml.cc 2013-03-07 04:22:27 PST ---
(In reply to comment #12)
> Commits pushed to master at https://github.com/D-Programming-Language/dmd
> 
> https://github.com/D-Programming-Language/dmd/commit/57b770ed49379c5af726d23356e0f75818a3f859
> Issue 7444 - Require [] for array copies too
> 
> https://github.com/D-Programming-Language/dmd/commit/ba1009c5561b51b8f18d9c869fde9bd45cb7ebc7
> Merge pull request #702 from 9rnsr/fix7444
> 
> Issue 7444 - Require [] for array copies too

I have tried this change, and now the first test case of this ER:


int[100] foo() {
   int[100] a;
   return a;
}
void main() {
   int[10_000] a, b;
   auto c = new int[10_000];
   a = 1;
   a = b;
   a = c;
   auto d = foo();
}



gives a ICE:

temp.d(8): Warning: explicit element-wise assignment (a)[] = 1 is better than a
= 1
temp.d(10): Warning: explicit element-wise assignment (a)[] = (c)[] is better
than a = c
Assertion failure: '0' on line 1193 in file 'glue.c'

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 07, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #14 from bearophile_hugs@eml.cc 2013-03-07 04:58:19 PST ---
(In reply to comment #13)

> gives a ICE:

Smaller test case:


void main() {
   int[1] a;
   a = 1;
}

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 07, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #15 from Kenji Hara <k.hara.pg@gmail.com> 2013-03-07 05:14:27 PST ---
(In reply to comment #14)
> (In reply to comment #13)
> 
> > gives a ICE:
> 
> Smaller test case:
> 
> 
> void main() {
>     int[1] a;
>     a = 1;
> }

What version and compiler switch do you use? I cannot reproduce the ICE.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 07, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #16 from bearophile_hugs@eml.cc 2013-03-07 12:10:52 PST ---
(In reply to comment #15)

> What version and compiler switch do you use? I cannot reproduce the ICE.

I am using the GIT head compiler, I have downloaded and compiled dmd few hours
ago, after this patch was merged.

I am on Windows 32 bit, and I have compiled the code with:

dmd -wi test.d

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 07, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444



--- Comment #17 from Kenji Hara <k.hara.pg@gmail.com> 2013-03-07 15:41:56 PST ---
(In reply to comment #16)
> (In reply to comment #15)
> 
> > What version and compiler switch do you use? I cannot reproduce the ICE.
> 
> I am using the GIT head compiler, I have downloaded and compiled dmd few hours
> ago, after this patch was merged.
> 
> I am on Windows 32 bit, and I have compiled the code with:
> 
> dmd -wi test.d

Thanks. -w option does not reproduce it, but -wi does.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 08, 2013
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444


Kenji Hara <k.hara.pg@gmail.com> changed:

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


--- Comment #18 from Kenji Hara <k.hara.pg@gmail.com> 2013-03-07 17:00:31 PST ---
(In reply to comment #17)
> (In reply to comment #16)
> > (In reply to comment #15)
> > 
> > > What version and compiler switch do you use? I cannot reproduce the ICE.
> > 
> > I am using the GIT head compiler, I have downloaded and compiled dmd few hours
> > ago, after this patch was merged.
> > 
> > I am on Windows 32 bit, and I have compiled the code with:
> > 
> > dmd -wi test.d
> 
> Thanks. -w option does not reproduce it, but -wi does.

OK, I opened a new report bug 9663 for the regression.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 20
[Issue 7444] Require [] for array copies too
http://d.puremagic.com/issues/show_bug.cgi?id=7444


Walter Bright <bugzilla@digitalmars.com> changed:

          What    |Removed                     |Added
----------------------------------------------------------------------------
        Resolution|FIXED                       |WONTFIX


--- Comment #19 from Walter Bright <bugzilla@digitalmars.com> 2013-10-20 09:38:57 PDT ---
This pull:

https://github.com/D-Programming-Language/dmd/pull/2673

undoes it with explanation. Reclassified as "wontfix".

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
1 2 3
Top | Discussion index | About this forum | D home