Thread overview
[Issue 9268] New: [ice-on-invalid] void assignment in fail44.d no longer caught in frontend
Jan 04, 2013
David Nadlinger
Jan 07, 2013
Kenji Hara
Jan 07, 2013
Kenji Hara
Jan 08, 2013
Kenji Hara
January 04, 2013
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: -------
January 07, 2013
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: -------
January 07, 2013
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: -------
January 07, 2013
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: -------
January 08, 2013
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: -------