November 20, 2012 [Issue 7444] Require [] for array copies too | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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 | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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, 2013 [Issue 7444] Require [] for array copies too | ||||
---|---|---|---|---|
| ||||
Posted in reply to bearophile_hugs@eml.cc | 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: ------- |
Copyright © 1999-2021 by the D Language Foundation