all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Eli Zaretskii <eliz@gnu.org>
Cc: rudalics@gmx.at, eggert@cs.ucla.edu, emacs-devel@gnu.org
Subject: Re: Tooltips GC overhead
Date: Fri, 07 Aug 2015 10:52:34 -0400	[thread overview]
Message-ID: <jwvzj24qa5x.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <83si7ztc6y.fsf@gnu.org> (Eli Zaretskii's message of "Tue, 04 Aug 2015 16:28:37 +0300")

> Just looking at the implementation of Fcons, I find it hard to believe
> a single comparison can have any significant effect on its efficiency.
> So I'm unsure how this could be of any practical concern.

Fcons is a core operation executed fairly frequently, so any slowdown
(no matter how small) needs to be justified by a major advantage.

Making the allocation profiler (which so far has never been useful to
anyone, AFAICT) slightly more precise, is not obviously a "major
advantage".  At the very least, I'd need positive proof that the extra
precision has been useful at least once.

>> > In any case, it's much better than the current useless profile.
>> I'm not convinced it'd make it sufficiently better
> Well, it allowed me in this case to identify the source of the problem
> quickly, efficiently, and accurately.

IIUC, something *else* allowed you to do that.  I don't think anyone
has actually tried to improve the precision of the allocation profiler
and found it to help in this case.
It's quite possible that some other problems might still prevent it from
being useful in this case.

>> I'd be OK with adding this "more precise allocation profiler" as
>> a compile-time option, so we could at least see if the improved
>> precision does make it more useful.
> IME, compile-time options of this kind are rarely useful, because they
> are never there when you need them.  Especially if the person who
> needs to perform measurements doesn't build their Emacs.

Until I have positive proof that the extra precision is useful in some
cases, I'll reject such changes in the default build.  So it's either
"rejected" or "depends on a compile-time option" for now.


        Stefan



  reply	other threads:[~2015-08-07 14:52 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-28 12:45 Tooltips GC overhead martin rudalics
2015-07-28 13:53 ` Eli Zaretskii
2015-07-28 15:09   ` martin rudalics
2015-07-28 23:51 ` Stefan Monnier
2015-07-29  7:18   ` martin rudalics
2015-07-29 14:29     ` Paul Eggert
2015-07-29 15:40       ` martin rudalics
2015-07-29 16:33         ` Eli Zaretskii
2015-07-29 18:05           ` martin rudalics
2015-07-29 21:15             ` Paul Eggert
2015-07-30  6:00               ` martin rudalics
2015-07-30  7:08               ` martin rudalics
2015-07-30  7:19                 ` Paul Eggert
2015-07-30  9:05                   ` martin rudalics
2015-07-30 16:36                     ` Eli Zaretskii
2015-07-30 19:53                       ` martin rudalics
2015-07-30 23:09                         ` Stefan Monnier
2015-07-30 23:33                           ` Drew Adams
2015-07-31  6:48                             ` Eli Zaretskii
2015-07-31  6:47                         ` Eli Zaretskii
2016-03-06  9:21                     ` martin rudalics
2016-03-08  8:02                       ` martin rudalics
2015-08-01 10:49               ` martin rudalics
2015-08-03 10:34                 ` Stefan Monnier
2015-08-03 14:56                   ` Eli Zaretskii
2015-08-03 21:13                     ` Stefan Monnier
2015-08-04  2:38                       ` Eli Zaretskii
2015-08-04  7:31                         ` Stefan Monnier
2015-08-04 13:28                           ` Eli Zaretskii
2015-08-07 14:52                             ` Stefan Monnier [this message]
2015-08-07 15:23                               ` Eli Zaretskii
2015-08-07 17:26                                 ` Stefan Monnier
2015-08-07 18:20                                   ` Eli Zaretskii
2015-08-07 21:25                                     ` Stefan Monnier
2015-08-08  6:46                                       ` Eli Zaretskii
2015-08-08  7:01                                         ` David Kastrup
2015-08-07 18:26                                 ` Paul Eggert
2015-08-07 18:59                                   ` Eli Zaretskii
2015-08-08 13:41                           ` Nix
2015-08-08 16:10                             ` Stefan Monnier
2015-08-08 16:13                               ` David Kastrup
2015-08-10 17:05                                 ` Nix
2015-08-10 17:44                                   ` David Kastrup
2015-07-29 16:45         ` Paul Eggert
2015-07-29 18:05           ` martin rudalics
2015-07-29 21:17             ` Paul Eggert
2015-07-30  6:00               ` martin rudalics

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=jwvzj24qa5x.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=eggert@cs.ucla.edu \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=rudalics@gmx.at \
    /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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.