public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, PR d/88722] Partially fix ICE: in register_moduleinfo, at d/modules.cc:40
@ 2019-08-21  8:02 Iain Buclaw
  0 siblings, 0 replies; only message in thread
From: Iain Buclaw @ 2019-08-21  8:02 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 656 bytes --]

Hi,

This patch turns an ICE into a sorry() error.  Not marking the PR as
fixed, as an implementation of ModuleInfo code generation should be
worked out when named section support is not present on the target.

Bootstrapped and regression tested the D testsuite on
x86_64-linux-gnu, with further checking done on pdp11-aout
cross-compiler target to verify the sorry() path is executed and the
ICE is no longer present.

Committed to trunk as r274769.

--
Iain
---
gcc/d/ChangeLog:

        PR d/88722
        * modules.cc: Include diagnostic.h.
        (register_moduleinfo): Use sorry instead of gcc_assert for targets
        without named sections.
---

[-- Attachment #2: pr88722.patch --]
[-- Type: application/x-patch, Size: 782 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2019-08-21  7:59 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-21  8:02 [PATCH, PR d/88722] Partially fix ICE: in register_moduleinfo, at d/modules.cc:40 Iain Buclaw

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).