[Bug d/96301] New: d: internal compiler error: Segmentation fault during RTL pass: expand on armhf/armel/s390x

[Bug d/96301] New: d: internal compiler error: Segmentation fault during RTL pass: expand on armhf/armel/s390x

[ibuclaw at gdcproject dot org]

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

            Bug ID: 96301
           Summary: d: internal compiler error: Segmentation fault during
                    RTL pass: expand on armhf/armel/s390x
           Product: gcc
           Version: 10.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: d
          Assignee: ibuclaw at gdcproject dot org
          Reporter: ibuclaw at gdcproject dot org
  Target Milestone: ---

Seen here https://buildd.debian.org/status/package.php?p=gtk-d&suite=sid

Reduced test, accommodating for struct and static array returns as well.

Compile with '-frelease -fPIC -O2' or '-O2 -fno-assert' to trigger
---
version (Struct)
{
    struct Type
    {
        size_t length;
        int* ptr;
    }
}
else version (Array)
{
    alias Type = int[100];
]
else
{
    alias Type = int[];
}

class Container
{
    Type children;

    void remove(void* data)
    {
        Type remove(Type range)
        {
            auto result = range;
            if (result.front)
                return result;
            assert(0);
        }
        if (data)
            remove(children);
    }
}

int front(Type a)
{
    return a.ptr[0];
}

[Bug d/96301] d: internal compiler error: Segmentation fault during RTL pass: expand on armhf/armel/s390x

[ibuclaw at gdcproject dot org]

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

--- Comment #1 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
The static array path has a different ICE, but the underlying cause is the
same.

test.d:20:18: internal compiler error: in convert_memory_address_addr_space_1,
at explow.c:302
   20 |             auto result = range;
      |                  ^
0x5f8abc convert_memory_address_addr_space_1(scalar_int_mode, rtx_def*,
unsigned char, bool, bool)
        ../../gcc/explow.c:302
0x5f8b2f convert_memory_address_addr_space_1(scalar_int_mode, rtx_def*,
unsigned char, bool, bool)
        ../../gcc/explow.c:405
0x5f8b2f convert_memory_address_addr_space(scalar_int_mode, rtx_def*, unsigned
char)
        ../../gcc/explow.c:404
0x5f8b2f memory_address_addr_space(machine_mode, rtx_def*, unsigned char)
        ../../gcc/explow.c:418
0x8f8171 expand_expr_real_1(tree_node*, rtx_def*, machine_mode,
expand_modifier, rtx_def**, bool)
        ../../gcc/expr.c:10462
0x902578 expand_expr
        ../../gcc/expr.h:282
0x902578 expand_assignment(tree_node*, tree_node*, bool)
        ../../gcc/expr.c:5158
0x902578 expand_assignment(tree_node*, tree_node*, bool)
        ../../gcc/expr.c:5056
0x7ff7a0 expand_gimple_stmt_1
        ../../gcc/cfgexpand.c:3749
0x7ff7a0 expand_gimple_stmt
        ../../gcc/cfgexpand.c:3847
0x804aff expand_gimple_basic_block
        ../../gcc/cfgexpand.c:5888
0x806f16 execute
        ../../gcc/cfgexpand.c:6543

[Bug d/96301] d: internal compiler error: Segmentation fault during RTL pass: expand on armhf/armel/s390x

[cvs-commit at gcc dot gnu.org]

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

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

https://gcc.gnu.org/g:6bebbc033d8bf2246745ffef7186b0424e08ba6b

commit r11-2734-g6bebbc033d8bf2246745ffef7186b0424e08ba6b
Author: Iain Buclaw <ibuclaw@gdcproject.org>
Date:   Fri Jul 24 13:49:37 2020 +0200

    d: Fix ICE Segmentation fault during RTL pass: expand on armhf/armel/s390x

    gcc/d/ChangeLog:

            PR d/96301
            * decl.cc (DeclVisitor::visit (FuncDeclaration *)): Only return
            non-trivial structs by invisible reference.

    gcc/testsuite/ChangeLog:

            PR d/96301
            * gdc.dg/pr96301a.d: New test.
            * gdc.dg/pr96301b.d: New test.
            * gdc.dg/pr96301c.d: New test.

[Bug d/96301] d: internal compiler error: Segmentation fault during RTL pass: expand on armhf/armel/s390x

[cvs-commit at gcc dot gnu.org]

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

--- Comment #3 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:412068819b463751ef476183077638b3429213de

commit r10-8637-g412068819b463751ef476183077638b3429213de
Author: Iain Buclaw <ibuclaw@gdcproject.org>
Date:   Fri Jul 24 13:49:37 2020 +0200

    d: Fix ICE Segmentation fault during RTL pass: expand on armhf/armel/s390x

    gcc/d/ChangeLog:

            PR d/96301
            * decl.cc (DeclVisitor::visit (FuncDeclaration *)): Only return
            non-trivial structs by invisible reference.

    gcc/testsuite/ChangeLog:

            PR d/96301
            * gdc.dg/pr96301a.d: New test.
            * gdc.dg/pr96301b.d: New test.
            * gdc.dg/pr96301c.d: New test.

    (cherry picked from commit 6bebbc033d8bf2246745ffef7186b0424e08ba6b)

[Bug d/96301] d: internal compiler error: Segmentation fault during RTL pass: expand on armhf/armel/s390x

[ibuclaw at gdcproject dot org]

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

Iain Buclaw <ibuclaw at gdcproject dot org> changed:

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

--- Comment #4 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
Done.