public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Florian Weimer <fweimer@redhat.com>
To: Thomas Neumann <tneumann@users.sourceforge.net>
Cc: Gcc <gcc@gcc.gnu.org>
Subject: Re: performance of exception handling
Date: Mon, 11 May 2020 16:22:39 +0200	[thread overview]
Message-ID: <87wo5id00g.fsf@oldenburg2.str.redhat.com> (raw)
In-Reply-To: <929cc727-20c6-480c-3412-d3f7a08f7544@users.sourceforge.net> (Thomas Neumann's message of "Mon, 11 May 2020 15:59:16 +0200")

* Thomas Neumann:

>> Link: <https://wg21.link/P0709>
>> 
>> I'm not sure if your summary is correct.
>
> I was referring to Section 3.2, where Herb says:
>
> "We must remove all technical reasons for a C++ project to disable
> exception handling (e.g., by compiler switch) or ban use of exceptions,
> in all or part of their project."

Ah, but I think his other papers make it clear that he wants a unified
error handling mechanism for C++, but something that is not based on
exceptions with stack unwinding.  Or has he changed his mind?

> In a way I am disagreeing with the paper, of course, in that I propose
> to make the existing exception mechanism faster instead of inventing a
> new exception mechanism. But what I agree on with P0709 is that it is
> unfortunate that many projects disable exceptions due to performance
> concerns. And I think the performance problems can be solved.

Yes, I think we agree.

>> My current preferred solution is something that moves the entire code
>> that locates the relevant FDE table into glibc.
>
> That is indeed an option, but I have two concerns there. First, it will
> lead to code duplication, as libgcc will have to continue to provide its
> on implementation on systems with "old" glibcs lacking
> __dl_ehframe_find.

That's true for any approach with backwards compatibility.

> And second, libgcc has a second lookup mechanism for
> __register_frame_info_bases etc., which is needed to JITed code anyway.
> And it seems to be attractive to handle that in the same data structure
> that also covers the code from executables and shared libraries.

That would still need dynamic linker changes because currently, it is
not possible to report failure from an ELF constructor, and registration
may fail due to the memory allocations.

> Of course one could move that part to glibc, too. But the code
> duplication problems will persist for a long time, as gcc cannot rely
> upon the system glibc being new enough to provide __dl_ehframe_find.

Sure.  I'm not too worried about the explicitly registration code.  We
can optimize that independently, with a fast path for the common case
that no frames have been registered using this mechanism.

Thanks,
Florian


  reply	other threads:[~2020-05-11 14:23 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-11  8:14 Thomas Neumann
2020-05-11 10:40 ` Florian Weimer
2020-05-11 13:59   ` Thomas Neumann
2020-05-11 14:22     ` Florian Weimer [this message]
2020-05-11 15:14     ` size of exception handling (Was: performance of exception handling) Moritz Strübe
2020-05-12  7:20       ` Freddie Chopin
2020-05-12  7:47         ` Oleg Endo
2020-05-13  9:13           ` Jonathan Wakely
2020-05-12  9:16         ` size of exception handling Florian Weimer
2020-05-12  9:44           ` Freddie Chopin
2020-05-12 11:11             ` Jonathan Wakely
2020-05-12 11:17             ` Moritz Strübe
2020-05-12 11:29               ` Florian Weimer
2020-05-12 12:01                 ` Moritz Strübe
2020-05-12 11:07         ` size of exception handling (Was: performance of exception handling) Jonathan Wakely
2020-05-12 20:56           ` Freddie Chopin
2020-05-12 22:39             ` Jonathan Wakely
2020-05-12 22:48               ` Jonathan Wakely
2020-05-13  8:04                 ` David Brown
2020-05-12  9:03       ` size of exception handling Florian Weimer
2020-05-11 14:36   ` performance " David Edelsohn
2020-05-11 14:52     ` Florian Weimer
2020-05-11 15:12       ` David Edelsohn
2020-05-11 15:24         ` Florian Weimer
2020-05-12  6:08     ` Thomas Neumann
2020-05-12  7:15       ` Richard Biener
2020-05-12  7:30         ` Thomas Neumann
2020-05-12  9:01       ` Richard Sandiford
2020-05-13  1:13         ` Thomas Neumann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87wo5id00g.fsf@oldenburg2.str.redhat.com \
    --to=fweimer@redhat.com \
    --cc=gcc@gcc.gnu.org \
    --cc=tneumann@users.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).