[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: master 1ca436a: Make compilation mode work with warnings from compil

From: Alan Mackenzie
Subject: Re: master 1ca436a: Make compilation mode work with warnings from compiled buffer functions
Date: Wed, 28 Nov 2018 20:16:00 +0000
User-agent: Mutt/1.10.1 (2018-07-13)

Hello, Glenn.

On Wed, Nov 28, 2018 at 11:37:55 -0500, Glenn Morris wrote:

> Hi,

> Alan Mackenzie wrote:

> > --- a/lisp/emacs-lisp/bytecomp.el
> > +++ b/lisp/emacs-lisp/bytecomp.el
> > @@ -124,6 +124,7 @@
> >  (require 'backquote)
> >  (require 'macroexp)
> >  (require 'cconv)
> > +(require 'compile)

> I would expect this extra require to slow down bootstrap, for an issue
> that is not relevant to bootstrap (bug#33475)?

I've looked at this, and I don't think there would be other than a
microscopic slowdown.  The previous version of bytecomp.el used at least
one function in compile.el (namely compilation-mode), suggesting that
compile.elc? is already loaded when bytecomp starts.  However, for some
fiddly reason a macro in compile.el was no longer defined, hence the

> Can this be avoided?

Yes it can, by moving the declaration of emacs-lisp-compilation-mode
out of bytecomp.el, probably into compile.el itself.

> Indeed, my automatic overnight build today was about 13 mins, whereas
> every previous day this month was < 11.5 mins.

I've timed several builds, and seen no differences between before and
after, appart from normal measurement error.  I see 2min 29sec +- 1
second for "cold starts", and 2min 3sec += 1 second for a subsequent
build, when all the source files are in filestore cache.

Are you sure that bug #33475 is the cause of your slowdown?

Alan Mackenzie (Nuremberg, Germany).

reply via email to

[Prev in Thread] Current Thread [Next in Thread]