[Bug d/97842] New: ice compiling dxml

[Bug d/97842] New: ice compiling dxml

[alex at sunopti dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

            Bug ID: 97842
           Summary: ice compiling dxml
           Product: gcc
           Version: 10.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: d
          Assignee: ibuclaw at gdcproject dot org
          Reporter: alex at sunopti dot com
  Target Milestone: ---

Created attachment 49566
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49566&action=edit
The source for dxml to repro the bug.

Internal compiler error when compiling dxml library, using gdc 10.2.0 on debian
testing.

Note : Dxml needs a modification to the standard library to expose source
property in std.util.ByCodeUnitImpl

sudo nano '/usr/lib/gcc/x86_64-linux-gnu/10/include/d/std/utf.d'
Insert at line 3333 :
auto source() const { return str; }

gdc -funittest dxml/parser.d

[Bug d/97842] ice compiling dxml

[ibuclaw at gdcproject dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

--- Comment #1 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
It would be great if you can use dustmite to make a reduced test case,
preferably by reducing phobos as well so it's just a standalone test.

https://github.com/CyberShadow/DustMite

[Bug d/97842] ice compiling dxml

[ibuclaw at gdcproject dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

--- Comment #2 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
I think it is this issue that is being hit:
https://issues.dlang.org/show_bug.cgi?id=18970

[Bug d/97842] ice compiling dxml

[ibuclaw at gdcproject dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

--- Comment #3 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
Upstream PR raised with related backports.

https://github.com/dlang/dmd/pull/11971

[Bug d/97842] ice compiling dxml

[alex at sunopti dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

--- Comment #4 from Alex <alex at sunopti dot com> ---
Created attachment 49573
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49573&action=edit
dustmite reduced problem.

[Bug d/97842] ice compiling dxml

[ibuclaw at gdcproject dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

--- Comment #5 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
(In reply to Alex from comment #4)
> Created attachment 49573 [details]
> dustmite reduced problem.

Your reduction seems to have instead found another segfault. :-)

https://issues.dlang.org/show_bug.cgi?id=21164

[Bug d/97842] ice compiling dxml

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Iain Buclaw
<ibuclaw@gcc.gnu.org>:

https://gcc.gnu.org/g:7e6dbe4e571a375330beb4266813006dc474b716

commit r10-9042-g7e6dbe4e571a375330beb4266813006dc474b716
Author: Iain Buclaw <ibuclaw@gdcproject.org>
Date:   Tue Nov 17 10:48:41 2020 +0100

    d: Fix a couple of ICEs found in the dmd front-end (PR97842)

    - Segmentation fault on incomplete static if.
    - Segmentation fault resolving typeof() expression when gagging is on.

    gcc/d/ChangeLog:

            PR d/97842
            * dmd/cond.c (StaticIfCondition::include): Return error if
condition
            expression is unset.
            * dmd/mtype.c (TypeTypeof::resolve): Return error if scope is
unset.

    gcc/testsuite/ChangeLog:

            PR d/97842
            * gdc.test/fail_compilation/fail18970.d: New test.
            * gdc.test/fail_compilation/imports/test21164a.d: New test.
            * gdc.test/fail_compilation/imports/test21164b.d: New test.
            * gdc.test/fail_compilation/imports/test21164c.d: New test.
            * gdc.test/fail_compilation/imports/test21164d.d: New test.
            * gdc.test/fail_compilation/test21164.d: New test.

    (cherry picked from commit 27d8c3516b67c0f5a8fe8970d0558ee3b97e8281)

[Bug d/97842] ice compiling dxml

[ibuclaw at gdcproject dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97842

Iain Buclaw <ibuclaw at gdcproject dot org> changed:

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

--- Comment #7 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
Fix for both segfaults committed to mainline and backported to gcc-10.