all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Tom Gillespie <tgbugs@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: Add a configure option for NATIVE_FULL_AOT?
Date: Wed, 18 Aug 2021 15:04:05 +0300	[thread overview]
Message-ID: <83a6lf558q.fsf@gnu.org> (raw)
In-Reply-To: <CA+G3_PM+B9NDx6Ff99kxyih+LG6QoJqgquuJDANFajqzf9ZZjg@mail.gmail.com> (message from Tom Gillespie on Tue, 17 Aug 2021 14:52:10 -0700)

> From: Tom Gillespie <tgbugs@gmail.com>
> Date: Tue, 17 Aug 2021 14:52:10 -0700
> Cc: Eli Zaretskii <eliz@gnu.org>
> 
> Short version: I think concerns about NATIVE_FULL_AOT are
> orthogonal from issues with having native comp enabled at all.

I agree (and never said anything to the contrary).  I was only talking
about the former, if it is treated as the main or even very important
paradigm.

> Indeed, however, the presence of the native-lisp/*/preloaded folder
> means that the
> concerns you raised in the other thread are present regardless of whether
> NATIVE_FULL_AOT is set. For example, trying to modify files.el and get those
> changes to propagate for an invocation of emacs -q is currently not possible due
> to fact that the /preloaded/ els will always be present and will take
> priority when
> native compilation is enabled (removing files-*.eln from preloaded will also
> leave Emacs in a completely broken state as I had the misfortune to learn).

Modification of preloaded Lisp files requires re-dumping Emacs,
always did.  In general, you cannot easily do that without having the
source tree and rebuilding there, not unless you know very well what
to do.  If you do know what to do, the presence of the *.eln files in
preloaded/ doesn't prevent you from rebuilding Emacs: you just need to
replace files*.eln with the newly-compiled one, and re-dump, which is
basically what happens during a re-build.

> One outstanding question is whether eln-cache always has priority
> over native-lisp

It does, except when Emacs knows the particular .eln file should be
put in preloaded/.

> > I didn't invest any thought in it, FWIW.  For me, it was always a "you
> > are on your own" option.  I didn't even try it, as all of the efforts
> > went into making the JIT compilation working well and reliably.
> 
> I think that that will pay off in either case because the underlying
> compilation process is identical (if confusingly named). JIT -> async
> AOT, and FULL AOT just skipping the async bit for the core and doing
> it all at the same time.

I wish it were that simple.  It isn't.  Adding natively-compiled files
raises quite a few issues that didn't exist before, and up to now,
when I thought about these issues, I always considered the JIT modus
operandi, and it alone -- if things sounded right with JIT, I
considered the issue closed.



  reply	other threads:[~2021-08-18 12:04 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-17 16:03 Add a configure option for NATIVE_FULL_AOT? Tom Gillespie
2021-08-17 17:13 ` Eli Zaretskii
2021-08-17 21:52   ` Tom Gillespie
2021-08-18 12:04     ` Eli Zaretskii [this message]
2021-08-18  0:33 ` Arthur Miller
  -- strict thread matches above, loose matches on Subject: below --
2021-08-17  8:53 Ulrich Mueller
2021-08-17 11:42 ` Eli Zaretskii
2021-08-17 11:56   ` Ulrich Mueller
2021-08-17 12:09     ` Lars Ingebrigtsen
2021-08-17 13:03       ` Eli Zaretskii
2021-08-18 14:44         ` Lars Ingebrigtsen
2021-08-17 12:53     ` Eli Zaretskii
2021-08-17 13:07       ` Arthur Miller
2021-08-17 15:32       ` Yuri D'Elia
2021-08-17 17:01         ` Eli Zaretskii
2021-08-17 17:12           ` Yuri D'Elia
2021-08-17 18:19             ` Eli Zaretskii
2021-08-17 18:33           ` Andreas Schwab
2021-08-17 18:42             ` Eli Zaretskii
2021-08-17 18:46               ` Andreas Schwab
2021-08-17 19:01                 ` Eli Zaretskii
2021-08-17 19:05                   ` Andreas Schwab
2021-08-17 19:09                     ` Eli Zaretskii
2021-08-17 19:36               ` Ulrich Mueller
2021-08-18  0:48                 ` Arthur Miller
2021-08-18  7:29                   ` Andreas Schwab
2021-08-18 15:43                     ` Yuri D'Elia
2021-08-19  0:57                       ` Arthur Miller
2021-08-19  7:04                         ` tomas
2021-08-19 21:17                           ` Arthur Miller
2021-08-20  7:20                             ` tomas
2021-08-20 12:06                               ` Arthur Miller
2021-08-20 13:13                                 ` tomas
2021-08-20 19:51                                   ` Arthur Miller
2021-08-20 20:06                                     ` tomas
2021-08-20 21:25                                       ` Arthur Miller
2021-08-21  6:44                                         ` tomas
2021-08-21 18:20                                           ` Arthur Miller
2021-08-19  7:13                         ` Eli Zaretskii
2021-08-19 21:01                           ` Arthur Miller
2021-08-18  2:23                 ` Eli Zaretskii
2021-08-18  4:53                   ` Tassilo Horn
2021-08-18 12:07                     ` Eli Zaretskii
2021-08-19  2:34                       ` Richard Stallman
2021-08-19  6:30                         ` tomas
2021-08-19  7:07                         ` Eli Zaretskii
2021-08-19  7:17                           ` Andreas Schwab
2021-08-19  7:46                             ` Eli Zaretskii
2021-08-19  7:27                           ` tomas
2021-08-19  8:09                             ` Eli Zaretskii
2021-08-19 10:05                               ` tomas
2021-08-19 10:51                                 ` Eli Zaretskii
2021-08-19 12:49                                   ` tomas
2021-08-19 12:52                                     ` Eli Zaretskii
2021-08-19 13:09                                       ` tomas
2021-08-18  7:04                   ` Ulrich Mueller
2021-08-18 12:12                     ` Eli Zaretskii
2021-08-18  7:33                   ` tomas
2021-08-18 12:14                     ` Eli Zaretskii
2021-08-18 13:32                       ` tomas
2021-08-18 13:45                         ` Eli Zaretskii
2021-08-18 16:22                           ` tomas
2021-08-18 16:26                             ` Eli Zaretskii
2021-08-18 16:34                               ` tomas
2021-08-18 16:43                                 ` Eli Zaretskii
2021-08-18 16:56                                   ` tomas
2021-08-18 17:12                                     ` Eli Zaretskii
2021-08-18 16:48                             ` Stefan Monnier
2021-08-18 17:00                               ` tomas
2021-08-18 17:17                                 ` Eli Zaretskii
2021-08-18 17:34                                   ` tomas
2021-08-18 19:43                                   ` Andrea Corallo via Emacs development discussions.
2021-08-19  1:19                                     ` Stefan Monnier
2021-08-19  7:11                                       ` Eli Zaretskii
2021-08-19  8:01                                         ` Andrea Corallo via Emacs development discussions.
2021-08-18 17:04                               ` Eli Zaretskii
2021-08-18 19:44                                 ` Andrea Corallo via Emacs development discussions.
2021-08-19  7:17                                   ` Eli Zaretskii
2021-08-19  7:52                                     ` Andrea Corallo via Emacs development discussions.
2021-08-18 14:11           ` Stefan Kangas
2021-08-18 15:54             ` Eli Zaretskii
2021-08-18 19:13           ` Gunnar Horrigmo
2021-08-18 19:24             ` Eli Zaretskii
2021-08-20  8:22               ` Gunnar Horrigmo
2021-08-20 10:47                 ` Eli Zaretskii
2021-08-20 13:06                   ` Gunnar Horrigmo

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=83a6lf558q.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=tgbugs@gmail.com \
    /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.