www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 9268] New: [ice-on-invalid] void assignment in fail44.d no longer caught in frontend

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9268

           Summary: [ice-on-invalid] void assignment in fail44.d no longer
                    caught in frontend
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Keywords: ice, ice-on-invalid-code
          Severity: regression
          Priority: P2
         Component: DMD
        AssignedTo: nobody puremagic.com
        ReportedBy: code klickverbot.at


--- Comment #0 from David Nadlinger <code klickverbot.at> 2013-01-03 20:11:04
PST ---
fail_compilation/fail44.d (as reproduced below) now fails with a backend ICE
instead of a nice "expression … is void and has no value" error:

---
void Foo()
{
  void[] bar;
  void[] foo;

  bar.length = 50;
  foo.length = 50;

  for(int i=0; i<50; i++)
  {
    foo[i] = bar[i];
  }
}
---

DMD 2.061:
---
Internal error: backend/cod4.c 359
---

Maybe d_do_test should generally check the error messages for "Internal error"
instead of just testing the exit value?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 03 2013
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9268


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull


--- Comment #1 from Kenji Hara <k.hara.pg gmail.com> 2013-01-07 05:44:16 PST ---
https://github.com/D-Programming-Language/dmd/pull/1434

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 07 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9268



--- Comment #2 from Kenji Hara <k.hara.pg gmail.com> 2013-01-07 06:30:43 PST ---
(In reply to comment #0)
 Maybe d_do_test should generally check the error messages for "Internal error"
 instead of just testing the exit value?

I added the code to check "Internal error: " message in the pull request. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Jan 07 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9268



--- Comment #3 from github-bugzilla puremagic.com 2013-01-07 10:05:54 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/97e936844677bedf0ab9a05739f63f843cddd155
fix Issue 9268 - [ice-on-invalid] void assignment in fail44.d no longer caught
in frontend

https://github.com/D-Programming-Language/dmd/commit/18e34e9d11f00d7078d13033d9845803eeb59027
Merge pull request #1434 from 9rnsr/fix9268

Issue 9268 - [ice-on-invalid] void assignment in fail44.d no longer caught in
frontend

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 07 2013
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9268


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

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


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