From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1873) id E8ABB387721F; Sun, 11 Dec 2022 18:00:25 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E8ABB387721F DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1670781625; bh=q0yk59olOaGWrZzkKSu8lbb4pnSrqP78Hmp0COJFX04=; h=From:To:Subject:Date:From; b=FLE6HtdXBuVIZBEZHhiuLDPQ6CgtGkdc9Bdlp4H6Bs9Ehgy4OOZKkQl78EvV1sxz2 eXtzNx3Jf9sSp/QY6MixvQlwGpH5oUiksRs8EtzHmXe/7X/S9xgrkgYg+CeExr/DAp aMddXYTL/L15FrJKp0wBHlopNz9h5T7rbXPXrvKo= MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Iain Buclaw To: gcc-cvs@gcc.gnu.org Subject: [gcc r11-10415] d: Fix internal compiler error: in visit, at d/imports.cc:72 (PR108050) X-Act-Checkin: gcc X-Git-Author: Iain Buclaw X-Git-Refname: refs/heads/releases/gcc-11 X-Git-Oldrev: 87c7e8a6cbdb8c08ae79ac2ff720fd59bbf16f9e X-Git-Newrev: 40b5a2354b58955aef168104c173311f062f1fa5 Message-Id: <20221211180025.E8ABB387721F@sourceware.org> Date: Sun, 11 Dec 2022 18:00:25 +0000 (GMT) List-Id: https://gcc.gnu.org/g:40b5a2354b58955aef168104c173311f062f1fa5 commit r11-10415-g40b5a2354b58955aef168104c173311f062f1fa5 Author: Iain Buclaw Date: Sat Dec 10 19:12:43 2022 +0100 d: Fix internal compiler error: in visit, at d/imports.cc:72 (PR108050) The visitor for lowering IMPORTED_DECLs did not have an override for dealing with importing OverloadSet symbols. This has now been implemented in the code generator. PR d/108050 gcc/d/ChangeLog: * decl.cc (DeclVisitor::visit (Import *)): Handle build_import_decl returning a TREE_LIST. * imports.cc (ImportVisitor::visit (OverloadSet *)): New override. gcc/testsuite/ChangeLog: * gdc.dg/imports/pr108050/mod1.d: New. * gdc.dg/imports/pr108050/mod2.d: New. * gdc.dg/imports/pr108050/package.d: New. * gdc.dg/pr108050.d: New test. (cherry picked from commit d9d8c9674ad3ad3aa38419d24b1aaaffe31f5d3f) Diff: --- gcc/d/decl.cc | 12 ++++++++++-- gcc/d/imports.cc | 14 ++++++++++++++ gcc/testsuite/gdc.dg/imports/pr108050/mod1.d | 2 ++ gcc/testsuite/gdc.dg/imports/pr108050/mod2.d | 2 ++ gcc/testsuite/gdc.dg/imports/pr108050/package.d | 2 ++ gcc/testsuite/gdc.dg/pr108050.d | 4 ++++ 6 files changed, 34 insertions(+), 2 deletions(-) diff --git a/gcc/d/decl.cc b/gcc/d/decl.cc index a30549ce8e0..fff7a7c2a74 100644 --- a/gcc/d/decl.cc +++ b/gcc/d/decl.cc @@ -197,8 +197,16 @@ public: tree name = (alias != NULL) ? get_identifier (alias->toChars ()) : NULL_TREE; - debug_hooks->imported_module_or_decl (decl, name, context, - false, false); + if (TREE_CODE (decl) != TREE_LIST) + debug_hooks->imported_module_or_decl (decl, name, context, + false, false); + else + { + /* Overload sets return a list of imported decls. */ + for (; decl != NULL_TREE; decl = TREE_CHAIN (decl)) + debug_hooks->imported_module_or_decl (TREE_VALUE (decl), name, + context, false, false); + } } } else diff --git a/gcc/d/imports.cc b/gcc/d/imports.cc index 2288843c61a..399c3ee3d25 100644 --- a/gcc/d/imports.cc +++ b/gcc/d/imports.cc @@ -157,6 +157,20 @@ public: } } + /* Build IMPORTED_DECLs for all overloads in a set. */ + void visit (OverloadSet *d) + { + vec *tset = NULL; + + vec_alloc (tset, d->a.length); + + for (size_t i = 0; i < d->a.length; i++) + vec_safe_push (tset, build_import_decl (d->a[i])); + + d->isym = build_tree_list_vec (tset); + tset->truncate (0); + } + /* Function aliases are the same as alias symbols. */ void visit (FuncAliasDeclaration *d) { diff --git a/gcc/testsuite/gdc.dg/imports/pr108050/mod1.d b/gcc/testsuite/gdc.dg/imports/pr108050/mod1.d new file mode 100644 index 00000000000..f27a13dc051 --- /dev/null +++ b/gcc/testsuite/gdc.dg/imports/pr108050/mod1.d @@ -0,0 +1,2 @@ +module imports.pr108050.mod1; +string[] split() { return null; } diff --git a/gcc/testsuite/gdc.dg/imports/pr108050/mod2.d b/gcc/testsuite/gdc.dg/imports/pr108050/mod2.d new file mode 100644 index 00000000000..29d8aa8f53e --- /dev/null +++ b/gcc/testsuite/gdc.dg/imports/pr108050/mod2.d @@ -0,0 +1,2 @@ +module imports.pr108050.mod2; +string[] split() { return null; } diff --git a/gcc/testsuite/gdc.dg/imports/pr108050/package.d b/gcc/testsuite/gdc.dg/imports/pr108050/package.d new file mode 100644 index 00000000000..b8b03b832af --- /dev/null +++ b/gcc/testsuite/gdc.dg/imports/pr108050/package.d @@ -0,0 +1,2 @@ +module imports.pr108050; +public import imports.pr108050.mod1, imports.pr108050.mod2; diff --git a/gcc/testsuite/gdc.dg/pr108050.d b/gcc/testsuite/gdc.dg/pr108050.d new file mode 100644 index 00000000000..69134e73137 --- /dev/null +++ b/gcc/testsuite/gdc.dg/pr108050.d @@ -0,0 +1,4 @@ +// { dg-do compile } +// { dg-additional-sources "imports/pr108050/package.d imports/pr108050/mod1.d imports/pr108050/mod2.d" } +// { dg-options "-g" } +import imports.pr108050 : split;