unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: David Kastrup <dak@gnu.org>
Cc: Andreas Schwab <schwab@suse.de>,
	emacs-devel@gnu.org, rms@gnu.org, rgm@gnu.org, storm@cua.dk
Subject: Re: cl-byte-compile-compiler-macro
Date: Wed, 05 Apr 2006 21:39:17 +0200	[thread overview]
Message-ID: <85acaz24nu.fsf@lola.goethe.zz> (raw)
In-Reply-To: <jwvvetnstv2.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Wed, 05 Apr 2006 15:32:18 -0400")

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>>> When I compile quux.el, does that need CL to be loaded?
>>> (Assume quux.el does not load CL itself.)
>
>>     I think it does, since the accessor function has a byte-compile property
>>     that points to function in cl-macs (cl-byte-compile-compiler-macro).
>
>> I think it should be possible to compile quux.el without loading CL.
>
>> I proposed moving that function and its subroutine to subr.el.
>> Will that achieve the goal?
>
> An alternative solution might be the following which just ignores the
> cl-byte-compile-compiler-macro value unless that function is indeed defined.
> Since compiler-macros are supposed to only provide optimizations but no
> change in semantics, no running them should always be harmless.
> Please install it if you think it's right.

Uh, what?  Am I just confused about the terminology you use?  defmacro
most certainly does not provide optional stuff to the compiler.
defsubst would, but is the result of that called a "compiler macro"
while the result of "defmacro" isn't?

Anyway, I don't even think that a defsubst which might or might not be
executed is a good idea: if the programmer wants to achieve a certain
effect with an optimization, failing to do so should not go unnoticed.

It appears that the current behavior makes it hard to track down the
problem.  I think the solution should be to create better diagnostics
for pinpointing the problem instead of removing them completely.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum

  reply	other threads:[~2006-04-05 19:39 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-22 21:02 cl-byte-compile-compiler-macro Kim F. Storm
2006-03-23 23:52 ` cl-byte-compile-compiler-macro Glenn Morris
2006-03-24 23:25   ` cl-byte-compile-compiler-macro Richard Stallman
2006-03-25  2:52     ` cl-byte-compile-compiler-macro Glenn Morris
2006-03-25  9:43       ` cl-byte-compile-compiler-macro Glenn Morris
2006-03-25 14:20       ` cl-byte-compile-compiler-macro Andreas Schwab
2006-03-28  6:39         ` cl-byte-compile-compiler-macro Glenn Morris
2006-03-25 15:26       ` cl-byte-compile-compiler-macro Richard Stallman
2006-03-25 20:55         ` cl-byte-compile-compiler-macro Kim F. Storm
2006-03-26 11:21           ` cl-byte-compile-compiler-macro Richard Stallman
2006-03-26 14:10             ` cl-byte-compile-compiler-macro Andreas Schwab
2006-03-27  2:07               ` cl-byte-compile-compiler-macro Stefan Monnier
2006-03-27  9:34                 ` cl-byte-compile-compiler-macro Andreas Schwab
2006-03-27 22:29                 ` cl-byte-compile-compiler-macro Richard Stallman
2006-03-27 23:44                   ` cl-byte-compile-compiler-macro Andreas Schwab
2006-03-28 19:33                     ` cl-byte-compile-compiler-macro Richard Stallman
2006-04-04 19:57                     ` cl-byte-compile-compiler-macro Richard Stallman
2006-04-04 20:37                       ` cl-byte-compile-compiler-macro Andreas Schwab
2006-04-05  3:47                         ` cl-byte-compile-compiler-macro Richard Stallman
2006-04-05  9:27                           ` cl-byte-compile-compiler-macro Andreas Schwab
2006-04-05 13:04                             ` cl-byte-compile-compiler-macro Stefan Monnier
2006-04-05 19:06                             ` cl-byte-compile-compiler-macro Richard Stallman
2006-04-05 19:32                               ` cl-byte-compile-compiler-macro Stefan Monnier
2006-04-05 19:39                                 ` David Kastrup [this message]
2006-04-05 20:12                                   ` cl-byte-compile-compiler-macro Stefan Monnier
2006-04-06  3:58                                 ` cl-byte-compile-compiler-macro Richard Stallman
2006-04-08 15:09                                   ` cl-byte-compile-compiler-macro Stefan Monnier
2006-04-05 21:23                               ` cl-byte-compile-compiler-macro Andreas Schwab
2006-04-04 20:50                       ` cl-byte-compile-compiler-macro Stefan Monnier
2006-03-28  6:52                   ` cl-byte-compile-compiler-macro Glenn Morris

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=85acaz24nu.fsf@lola.goethe.zz \
    --to=dak@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=rgm@gnu.org \
    --cc=rms@gnu.org \
    --cc=schwab@suse.de \
    --cc=storm@cua.dk \
    /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 public inbox

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

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).