Mercurial > emacs
annotate lisp/emacs-lisp/byte-opt.el @ 111381:7cb97499fd27
Fix misleading Changelog entry.
author | Michael Albinus <michael.albinus@gmx.de> |
---|---|
date | Sun, 07 Nov 2010 08:02:15 +0100 |
parents | cda2045a5ee8 |
children | b8fde5ef9e14 376148b31b5e |
rev | line source |
---|---|
38412
253f761ad37b
Some fixes to follow coding conventions in files maintained by FSF.
Pavel Janík <Pavel@Janik.cz>
parents:
37908
diff
changeset
|
1 ;;; byte-opt.el --- the optimization passes of the emacs-lisp byte compiler |
848 | 2 |
104914
1a8afcc5fb20
(degrees-to-radians): Mark as free from side effects.
Glenn Morris <rgm@gnu.org>
parents:
100908
diff
changeset
|
3 ;; Copyright (C) 1991, 1994, 2000, 2001, 2002, 2003, 2004, 2005, 2006, |
106815 | 4 ;; 2007, 2008, 2009, 2010 Free Software Foundation, Inc. |
848 | 5 |
6 ;; Author: Jamie Zawinski <jwz@lucid.com> | |
7 ;; Hallvard Furuseth <hbf@ulrik.uio.no> | |
27823 | 8 ;; Maintainer: FSF |
848 | 9 ;; Keywords: internal |
757 | 10 |
11 ;; This file is part of GNU Emacs. | |
12 | |
94655
90a2847062be
Switch to recommended form of GPLv3 permissions notice.
Glenn Morris <rgm@gnu.org>
parents:
93209
diff
changeset
|
13 ;; GNU Emacs is free software: you can redistribute it and/or modify |
757 | 14 ;; it under the terms of the GNU General Public License as published by |
94655
90a2847062be
Switch to recommended form of GPLv3 permissions notice.
Glenn Morris <rgm@gnu.org>
parents:
93209
diff
changeset
|
15 ;; the Free Software Foundation, either version 3 of the License, or |
90a2847062be
Switch to recommended form of GPLv3 permissions notice.
Glenn Morris <rgm@gnu.org>
parents:
93209
diff
changeset
|
16 ;; (at your option) any later version. |
757 | 17 |
18 ;; GNU Emacs is distributed in the hope that it will be useful, | |
19 ;; but WITHOUT ANY WARRANTY; without even the implied warranty of | |
20 ;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
21 ;; GNU General Public License for more details. | |
22 | |
23 ;; You should have received a copy of the GNU General Public License | |
94655
90a2847062be
Switch to recommended form of GPLv3 permissions notice.
Glenn Morris <rgm@gnu.org>
parents:
93209
diff
changeset
|
24 ;; along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>. |
757 | 25 |
848 | 26 ;;; Commentary: |
27 | |
14169 | 28 ;; ======================================================================== |
29 ;; "No matter how hard you try, you can't make a racehorse out of a pig. | |
30 ;; You can, however, make a faster pig." | |
31 ;; | |
82790
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
32 ;; Or, to put it another way, the Emacs byte compiler is a VW Bug. This code |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
33 ;; makes it be a VW Bug with fuel injection and a turbocharger... You're |
14169 | 34 ;; still not going to make it go faster than 70 mph, but it might be easier |
35 ;; to get it there. | |
36 ;; | |
757 | 37 |
14169 | 38 ;; TO DO: |
39 ;; | |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
40 ;; (apply (lambda (x &rest y) ...) 1 (foo)) |
14169 | 41 ;; |
42 ;; maintain a list of functions known not to access any global variables | |
43 ;; (actually, give them a 'dynamically-safe property) and then | |
44 ;; (let ( v1 v2 ... vM vN ) <...dynamically-safe...> ) ==> | |
45 ;; (let ( v1 v2 ... vM ) vN <...dynamically-safe...> ) | |
46 ;; by recursing on this, we might be able to eliminate the entire let. | |
47 ;; However certain variables should never have their bindings optimized | |
48 ;; away, because they affect everything. | |
49 ;; (put 'debug-on-error 'binding-is-magic t) | |
50 ;; (put 'debug-on-abort 'binding-is-magic t) | |
51 ;; (put 'debug-on-next-call 'binding-is-magic t) | |
52 ;; (put 'inhibit-quit 'binding-is-magic t) | |
53 ;; (put 'quit-flag 'binding-is-magic t) | |
54 ;; (put 't 'binding-is-magic t) | |
55 ;; (put 'nil 'binding-is-magic t) | |
56 ;; possibly also | |
57 ;; (put 'gc-cons-threshold 'binding-is-magic t) | |
58 ;; (put 'track-mouse 'binding-is-magic t) | |
59 ;; others? | |
60 ;; | |
61 ;; Simple defsubsts often produce forms like | |
62 ;; (let ((v1 (f1)) (v2 (f2)) ...) | |
63 ;; (FN v1 v2 ...)) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
64 ;; It would be nice if we could optimize this to |
14169 | 65 ;; (FN (f1) (f2) ...) |
66 ;; but we can't unless FN is dynamically-safe (it might be dynamically | |
67 ;; referring to the bindings that the lambda arglist established.) | |
68 ;; One of the uncountable lossages introduced by dynamic scope... | |
69 ;; | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
70 ;; Maybe there should be a control-structure that says "turn on |
14169 | 71 ;; fast-and-loose type-assumptive optimizations here." Then when |
72 ;; we see a form like (car foo) we can from then on assume that | |
73 ;; the variable foo is of type cons, and optimize based on that. | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
74 ;; But, this won't win much because of (you guessed it) dynamic |
14169 | 75 ;; scope. Anything down the stack could change the value. |
76 ;; (Another reason it doesn't work is that it is perfectly valid | |
77 ;; to call car with a null argument.) A better approach might | |
78 ;; be to allow type-specification of the form | |
79 ;; (put 'foo 'arg-types '(float (list integer) dynamic)) | |
80 ;; (put 'foo 'result-type 'bool) | |
81 ;; It should be possible to have these types checked to a certain | |
82 ;; degree. | |
83 ;; | |
84 ;; collapse common subexpressions | |
85 ;; | |
86 ;; It would be nice if redundant sequences could be factored out as well, | |
87 ;; when they are known to have no side-effects: | |
88 ;; (list (+ a b c) (+ a b c)) --> a b add c add dup list-2 | |
89 ;; but beware of traps like | |
90 ;; (cons (list x y) (list x y)) | |
91 ;; | |
92 ;; Tail-recursion elimination is not really possible in Emacs Lisp. | |
93 ;; Tail-recursion elimination is almost always impossible when all variables | |
94 ;; have dynamic scope, but given that the "return" byteop requires the | |
95 ;; binding stack to be empty (rather than emptying it itself), there can be | |
96 ;; no truly tail-recursive Emacs Lisp functions that take any arguments or | |
97 ;; make any bindings. | |
98 ;; | |
99 ;; Here is an example of an Emacs Lisp function which could safely be | |
100 ;; byte-compiled tail-recursively: | |
101 ;; | |
102 ;; (defun tail-map (fn list) | |
103 ;; (cond (list | |
104 ;; (funcall fn (car list)) | |
105 ;; (tail-map fn (cdr list))))) | |
106 ;; | |
107 ;; However, if there was even a single let-binding around the COND, | |
108 ;; it could not be byte-compiled, because there would be an "unbind" | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
109 ;; byte-op between the final "call" and "return." Adding a |
14169 | 110 ;; Bunbind_all byteop would fix this. |
111 ;; | |
112 ;; (defun foo (x y z) ... (foo a b c)) | |
113 ;; ... (const foo) (varref a) (varref b) (varref c) (call 3) END: (return) | |
114 ;; ... (varref a) (varbind x) (varref b) (varbind y) (varref c) (varbind z) (goto 0) END: (unbind-all) (return) | |
115 ;; ... (varref a) (varset x) (varref b) (varset y) (varref c) (varset z) (goto 0) END: (return) | |
116 ;; | |
117 ;; this also can be considered tail recursion: | |
118 ;; | |
119 ;; ... (const foo) (varref a) (call 1) (goto X) ... X: (return) | |
120 ;; could generalize this by doing the optimization | |
121 ;; (goto X) ... X: (return) --> (return) | |
122 ;; | |
123 ;; But this doesn't solve all of the problems: although by doing tail- | |
124 ;; recursion elimination in this way, the call-stack does not grow, the | |
125 ;; binding-stack would grow with each recursive step, and would eventually | |
126 ;; overflow. I don't believe there is any way around this without lexical | |
127 ;; scope. | |
128 ;; | |
129 ;; Wouldn't it be nice if Emacs Lisp had lexical scope. | |
130 ;; | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
131 ;; Idea: the form (lexical-scope) in a file means that the file may be |
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
132 ;; compiled lexically. This proclamation is file-local. Then, within |
14169 | 133 ;; that file, "let" would establish lexical bindings, and "let-dynamic" |
134 ;; would do things the old way. (Or we could use CL "declare" forms.) | |
135 ;; We'd have to notice defvars and defconsts, since those variables should | |
136 ;; always be dynamic, and attempting to do a lexical binding of them | |
137 ;; should simply do a dynamic binding instead. | |
138 ;; But! We need to know about variables that were not necessarily defvarred | |
139 ;; in the file being compiled (doing a boundp check isn't good enough.) | |
140 ;; Fdefvar() would have to be modified to add something to the plist. | |
141 ;; | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
142 ;; A major disadvantage of this scheme is that the interpreter and compiler |
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
143 ;; would have different semantics for files compiled with (dynamic-scope). |
14169 | 144 ;; Since this would be a file-local optimization, there would be no way to |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
145 ;; modify the interpreter to obey this (unless the loader was hacked |
14169 | 146 ;; in some grody way, but that's a really bad idea.) |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
147 |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
148 ;; Other things to consider: |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
149 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
150 ;; ;; Associative math should recognize subcalls to identical function: |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
151 ;; (disassemble (lambda (x) (+ (+ (foo) 1) (+ (bar) 2)))) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
152 ;; ;; This should generate the same as (1+ x) and (1- x) |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
153 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
154 ;; (disassemble (lambda (x) (cons (+ x 1) (- x 1)))) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
155 ;; ;; An awful lot of functions always return a non-nil value. If they're |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
156 ;; ;; error free also they may act as true-constants. |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
157 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
158 ;; (disassemble (lambda (x) (and (point) (foo)))) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
159 ;; ;; When |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
160 ;; ;; - all but one arguments to a function are constant |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
161 ;; ;; - the non-constant argument is an if-expression (cond-expression?) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
162 ;; ;; then the outer function can be distributed. If the guarding |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
163 ;; ;; condition is side-effect-free [assignment-free] then the other |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
164 ;; ;; arguments may be any expressions. Since, however, the code size |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
165 ;; ;; can increase this way they should be "simple". Compare: |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
166 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
167 ;; (disassemble (lambda (x) (eq (if (point) 'a 'b) 'c))) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
168 ;; (disassemble (lambda (x) (if (point) (eq 'a 'c) (eq 'b 'c)))) |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
169 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
170 ;; ;; (car (cons A B)) -> (prog1 A B) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
171 ;; (disassemble (lambda (x) (car (cons (foo) 42)))) |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
172 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
173 ;; ;; (cdr (cons A B)) -> (progn A B) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
174 ;; (disassemble (lambda (x) (cdr (cons 42 (foo))))) |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
175 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
176 ;; ;; (car (list A B ...)) -> (prog1 A B ...) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
177 ;; (disassemble (lambda (x) (car (list (foo) 42 (bar))))) |
63625
9be013ed306d
(byte-optimize-pure-func): Fix spellings.
Juanma Barranquero <lekktu@gmail.com>
parents:
58490
diff
changeset
|
178 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
179 ;; ;; (cdr (list A B ...)) -> (progn A (list B ...)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
180 ;; (disassemble (lambda (x) (cdr (list 42 (foo) (bar))))) |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
181 |
757 | 182 |
848 | 183 ;;; Code: |
757 | 184 |
23822
34fa38b26638
Require bytecomp for byte-goto-ops.
Markus Rost <rost@math.uni-bielefeld.de>
parents:
23177
diff
changeset
|
185 (require 'bytecomp) |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
186 (eval-when-compile (require 'cl)) |
23822
34fa38b26638
Require bytecomp for byte-goto-ops.
Markus Rost <rost@math.uni-bielefeld.de>
parents:
23177
diff
changeset
|
187 |
757 | 188 (defun byte-compile-log-lap-1 (format &rest args) |
189 (if (aref byte-code-vector 0) | |
38412
253f761ad37b
Some fixes to follow coding conventions in files maintained by FSF.
Pavel Janík <Pavel@Janik.cz>
parents:
37908
diff
changeset
|
190 (error "The old version of the disassembler is loaded. Reload new-bytecomp as well")) |
757 | 191 (byte-compile-log-1 |
192 (apply 'format format | |
193 (let (c a) | |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
194 (mapcar (lambda (arg) |
757 | 195 (if (not (consp arg)) |
196 (if (and (symbolp arg) | |
197 (string-match "^byte-" (symbol-name arg))) | |
198 (intern (substring (symbol-name arg) 5)) | |
199 arg) | |
200 (if (integerp (setq c (car arg))) | |
201 (error "non-symbolic byte-op %s" c)) | |
202 (if (eq c 'TAG) | |
203 (setq c arg) | |
204 (setq a (cond ((memq c byte-goto-ops) | |
205 (car (cdr (cdr arg)))) | |
206 ((memq c byte-constref-ops) | |
207 (car (cdr arg))) | |
208 (t (cdr arg)))) | |
209 (setq c (symbol-name c)) | |
210 (if (string-match "^byte-." c) | |
211 (setq c (intern (substring c 5))))) | |
212 (if (eq c 'constant) (setq c 'const)) | |
213 (if (and (eq (cdr arg) 0) | |
214 (not (memq c '(unbind call const)))) | |
215 c | |
216 (format "(%s %s)" c a)))) | |
217 args))))) | |
218 | |
219 (defmacro byte-compile-log-lap (format-string &rest args) | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
220 `(and (memq byte-optimize-log '(t byte)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
221 (byte-compile-log-lap-1 ,format-string ,@args))) |
757 | 222 |
223 | |
224 ;;; byte-compile optimizers to support inlining | |
225 | |
226 (put 'inline 'byte-optimizer 'byte-optimize-inline-handler) | |
227 | |
228 (defun byte-optimize-inline-handler (form) | |
229 "byte-optimize-handler for the `inline' special-form." | |
230 (cons 'progn | |
231 (mapcar | |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
232 (lambda (sexp) |
58210
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
233 (let ((f (car-safe sexp))) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
234 (if (and (symbolp f) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
235 (or (cdr (assq f byte-compile-function-environment)) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
236 (not (or (not (fboundp f)) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
237 (cdr (assq f byte-compile-macro-environment)) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
238 (and (consp (setq f (symbol-function f))) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
239 (eq (car f) 'macro)) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
240 (subrp f))))) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
241 (byte-compile-inline-expand sexp) |
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
242 sexp))) |
757 | 243 (cdr form)))) |
244 | |
245 | |
767
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
246 ;; Splice the given lap code into the current instruction stream. |
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
247 ;; If it has any labels in it, you're responsible for making sure there |
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
248 ;; are no collisions, and that byte-compile-tag-number is reasonable |
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
249 ;; after this is spliced in. The provided list is destroyed. |
757 | 250 (defun byte-inline-lapcode (lap) |
251 (setq byte-compile-output (nconc (nreverse lap) byte-compile-output))) | |
252 | |
253 (defun byte-compile-inline-expand (form) | |
254 (let* ((name (car form)) | |
255 (fn (or (cdr (assq name byte-compile-function-environment)) | |
256 (and (fboundp name) (symbol-function name))))) | |
257 (if (null fn) | |
258 (progn | |
39770 | 259 (byte-compile-warn "attempt to inline `%s' before it was defined" |
28423
6b92fbc04c23
(byte-compile-inline-expand): Fix bug
Gerd Moellmann <gerd@gnu.org>
parents:
27823
diff
changeset
|
260 name) |
757 | 261 form) |
262 ;; else | |
28440
45337208670b
(byte-compile-inline-expand): Look
Gerd Moellmann <gerd@gnu.org>
parents:
28423
diff
changeset
|
263 (when (and (consp fn) (eq (car fn) 'autoload)) |
37907
a8fffaec8725
(byte-compile-inline-expand): Fix the arg of `load' again.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
37904
diff
changeset
|
264 (load (nth 1 fn)) |
28440
45337208670b
(byte-compile-inline-expand): Look
Gerd Moellmann <gerd@gnu.org>
parents:
28423
diff
changeset
|
265 (setq fn (or (and (fboundp name) (symbol-function name)) |
45337208670b
(byte-compile-inline-expand): Look
Gerd Moellmann <gerd@gnu.org>
parents:
28423
diff
changeset
|
266 (cdr (assq name byte-compile-function-environment))))) |
757 | 267 (if (and (consp fn) (eq (car fn) 'autoload)) |
37908
ba28e6b7a67b
(byte-compile-inline-expand): Complete Dave's
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
37907
diff
changeset
|
268 (error "File `%s' didn't define `%s'" (nth 1 fn) name)) |
58165
cc9f1e89279b
(byte-compile-inline-expand): Understand the
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
57423
diff
changeset
|
269 (if (and (symbolp fn) (not (eq fn t))) |
757 | 270 (byte-compile-inline-expand (cons fn (cdr form))) |
1818
7e3322619e46
compiled-function-p has been renamed to byte-code-function-p.
Jim Blandy <jimb@redhat.com>
parents:
957
diff
changeset
|
271 (if (byte-code-function-p fn) |
20778
6d7fffd02e26
(byte-compile-inline-expand): Use string-as-unibyte, if it is defined.
Richard M. Stallman <rms@gnu.org>
parents:
20749
diff
changeset
|
272 (let (string) |
11203
31b8e30c1afb
(byte-compile-inline-expand): Fetch actual bytecode
Karl Heuer <kwzh@gnu.org>
parents:
8466
diff
changeset
|
273 (fetch-bytecode fn) |
20778
6d7fffd02e26
(byte-compile-inline-expand): Use string-as-unibyte, if it is defined.
Richard M. Stallman <rms@gnu.org>
parents:
20749
diff
changeset
|
274 (setq string (aref fn 1)) |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
275 ;; Isn't it an error for `string' not to be unibyte?? --stef |
20778
6d7fffd02e26
(byte-compile-inline-expand): Use string-as-unibyte, if it is defined.
Richard M. Stallman <rms@gnu.org>
parents:
20749
diff
changeset
|
276 (if (fboundp 'string-as-unibyte) |
6d7fffd02e26
(byte-compile-inline-expand): Use string-as-unibyte, if it is defined.
Richard M. Stallman <rms@gnu.org>
parents:
20749
diff
changeset
|
277 (setq string (string-as-unibyte string))) |
79453 | 278 ;; `byte-compile-splice-in-already-compiled-code' |
279 ;; takes care of inlining the body. | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
280 (cons `(lambda ,(aref fn 0) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
281 (byte-code ,string ,(aref fn 2) ,(aref fn 3))) |
11203
31b8e30c1afb
(byte-compile-inline-expand): Fetch actual bytecode
Karl Heuer <kwzh@gnu.org>
parents:
8466
diff
changeset
|
282 (cdr form))) |
23177 | 283 (if (eq (car-safe fn) 'lambda) |
284 (cons fn (cdr form)) | |
285 ;; Give up on inlining. | |
286 form)))))) | |
757 | 287 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
288 ;; ((lambda ...) ...) |
757 | 289 (defun byte-compile-unfold-lambda (form &optional name) |
290 (or name (setq name "anonymous lambda")) | |
291 (let ((lambda (car form)) | |
292 (values (cdr form))) | |
1818
7e3322619e46
compiled-function-p has been renamed to byte-code-function-p.
Jim Blandy <jimb@redhat.com>
parents:
957
diff
changeset
|
293 (if (byte-code-function-p lambda) |
957 | 294 (setq lambda (list 'lambda (aref lambda 0) |
295 (list 'byte-code (aref lambda 1) | |
296 (aref lambda 2) (aref lambda 3))))) | |
757 | 297 (let ((arglist (nth 1 lambda)) |
298 (body (cdr (cdr lambda))) | |
299 optionalp restp | |
300 bindings) | |
301 (if (and (stringp (car body)) (cdr body)) | |
302 (setq body (cdr body))) | |
303 (if (and (consp (car body)) (eq 'interactive (car (car body)))) | |
304 (setq body (cdr body))) | |
305 (while arglist | |
306 (cond ((eq (car arglist) '&optional) | |
307 ;; ok, I'll let this slide because funcall_lambda() does... | |
308 ;; (if optionalp (error "multiple &optional keywords in %s" name)) | |
309 (if restp (error "&optional found after &rest in %s" name)) | |
310 (if (null (cdr arglist)) | |
311 (error "nothing after &optional in %s" name)) | |
312 (setq optionalp t)) | |
313 ((eq (car arglist) '&rest) | |
314 ;; ...but it is by no stretch of the imagination a reasonable | |
315 ;; thing that funcall_lambda() allows (&rest x y) and | |
316 ;; (&rest x &optional y) in arglists. | |
317 (if (null (cdr arglist)) | |
318 (error "nothing after &rest in %s" name)) | |
319 (if (cdr (cdr arglist)) | |
320 (error "multiple vars after &rest in %s" name)) | |
321 (setq restp t)) | |
322 (restp | |
323 (setq bindings (cons (list (car arglist) | |
324 (and values (cons 'list values))) | |
325 bindings) | |
326 values nil)) | |
327 ((and (not optionalp) (null values)) | |
39770 | 328 (byte-compile-warn "attempt to open-code `%s' with too few arguments" name) |
757 | 329 (setq arglist nil values 'too-few)) |
330 (t | |
331 (setq bindings (cons (list (car arglist) (car values)) | |
332 bindings) | |
333 values (cdr values)))) | |
334 (setq arglist (cdr arglist))) | |
335 (if values | |
336 (progn | |
337 (or (eq values 'too-few) | |
338 (byte-compile-warn | |
39770 | 339 "attempt to open-code `%s' with too many arguments" name)) |
757 | 340 form) |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
341 |
33458
f1c913d16f4c
(byte-compile-unfold-lambda): Don't
Gerd Moellmann <gerd@gnu.org>
parents:
32078
diff
changeset
|
342 ;; The following leads to infinite recursion when loading a |
f1c913d16f4c
(byte-compile-unfold-lambda): Don't
Gerd Moellmann <gerd@gnu.org>
parents:
32078
diff
changeset
|
343 ;; file containing `(defsubst f () (f))', and then trying to |
f1c913d16f4c
(byte-compile-unfold-lambda): Don't
Gerd Moellmann <gerd@gnu.org>
parents:
32078
diff
changeset
|
344 ;; byte-compile that file. |
f1c913d16f4c
(byte-compile-unfold-lambda): Don't
Gerd Moellmann <gerd@gnu.org>
parents:
32078
diff
changeset
|
345 ;(setq body (mapcar 'byte-optimize-form body))) |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
346 |
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
347 (let ((newform |
757 | 348 (if bindings |
349 (cons 'let (cons (nreverse bindings) body)) | |
350 (cons 'progn body)))) | |
351 (byte-compile-log " %s\t==>\t%s" form newform) | |
352 newform))))) | |
353 | |
354 | |
355 ;;; implementing source-level optimizers | |
356 | |
357 (defun byte-optimize-form-code-walker (form for-effect) | |
358 ;; | |
359 ;; For normal function calls, We can just mapcar the optimizer the cdr. But | |
360 ;; we need to have special knowledge of the syntax of the special forms | |
361 ;; like let and defun (that's why they're special forms :-). (Actually, | |
362 ;; the important aspect is that they are subrs that don't evaluate all of | |
363 ;; their args.) | |
364 ;; | |
365 (let ((fn (car-safe form)) | |
366 tmp) | |
367 (cond ((not (consp form)) | |
368 (if (not (and for-effect | |
369 (or byte-compile-delete-errors | |
370 (not (symbolp form)) | |
371 (eq form t)))) | |
372 form)) | |
373 ((eq fn 'quote) | |
374 (if (cdr (cdr form)) | |
39770 | 375 (byte-compile-warn "malformed quote form: `%s'" |
757 | 376 (prin1-to-string form))) |
377 ;; map (quote nil) to nil to simplify optimizer logic. | |
378 ;; map quoted constants to nil if for-effect (just because). | |
379 (and (nth 1 form) | |
380 (not for-effect) | |
381 form)) | |
1818
7e3322619e46
compiled-function-p has been renamed to byte-code-function-p.
Jim Blandy <jimb@redhat.com>
parents:
957
diff
changeset
|
382 ((or (byte-code-function-p fn) |
757 | 383 (eq 'lambda (car-safe fn))) |
110625
0e33b506a2f1
* emacs-lisp/byte-opt.el (byte-optimize-form-code-walker): Avoid
Andreas Schwab <schwab@linux-m68k.org>
parents:
106815
diff
changeset
|
384 (let ((newform (byte-compile-unfold-lambda form))) |
0e33b506a2f1
* emacs-lisp/byte-opt.el (byte-optimize-form-code-walker): Avoid
Andreas Schwab <schwab@linux-m68k.org>
parents:
106815
diff
changeset
|
385 (if (eq newform form) |
110987
cda2045a5ee8
Fix typos in docstrings, comments and ChangeLogs.
Juanma Barranquero <lekktu@gmail.com>
parents:
110625
diff
changeset
|
386 ;; Some error occurred, avoid infinite recursion |
110625
0e33b506a2f1
* emacs-lisp/byte-opt.el (byte-optimize-form-code-walker): Avoid
Andreas Schwab <schwab@linux-m68k.org>
parents:
106815
diff
changeset
|
387 form |
0e33b506a2f1
* emacs-lisp/byte-opt.el (byte-optimize-form-code-walker): Avoid
Andreas Schwab <schwab@linux-m68k.org>
parents:
106815
diff
changeset
|
388 (byte-optimize-form-code-walker newform for-effect)))) |
757 | 389 ((memq fn '(let let*)) |
390 ;; recursively enter the optimizer for the bindings and body | |
391 ;; of a let or let*. This for depth-firstness: forms that | |
392 ;; are more deeply nested are optimized first. | |
393 (cons fn | |
394 (cons | |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
395 (mapcar (lambda (binding) |
757 | 396 (if (symbolp binding) |
397 binding | |
398 (if (cdr (cdr binding)) | |
39770 | 399 (byte-compile-warn "malformed let binding: `%s'" |
757 | 400 (prin1-to-string binding))) |
401 (list (car binding) | |
402 (byte-optimize-form (nth 1 binding) nil)))) | |
403 (nth 1 form)) | |
404 (byte-optimize-body (cdr (cdr form)) for-effect)))) | |
405 ((eq fn 'cond) | |
406 (cons fn | |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
407 (mapcar (lambda (clause) |
757 | 408 (if (consp clause) |
409 (cons | |
410 (byte-optimize-form (car clause) nil) | |
411 (byte-optimize-body (cdr clause) for-effect)) | |
39770 | 412 (byte-compile-warn "malformed cond form: `%s'" |
757 | 413 (prin1-to-string clause)) |
414 clause)) | |
415 (cdr form)))) | |
416 ((eq fn 'progn) | |
417 ;; as an extra added bonus, this simplifies (progn <x>) --> <x> | |
418 (if (cdr (cdr form)) | |
419 (progn | |
420 (setq tmp (byte-optimize-body (cdr form) for-effect)) | |
421 (if (cdr tmp) (cons 'progn tmp) (car tmp))) | |
422 (byte-optimize-form (nth 1 form) for-effect))) | |
423 ((eq fn 'prog1) | |
424 (if (cdr (cdr form)) | |
425 (cons 'prog1 | |
426 (cons (byte-optimize-form (nth 1 form) for-effect) | |
427 (byte-optimize-body (cdr (cdr form)) t))) | |
428 (byte-optimize-form (nth 1 form) for-effect))) | |
429 ((eq fn 'prog2) | |
430 (cons 'prog2 | |
431 (cons (byte-optimize-form (nth 1 form) t) | |
432 (cons (byte-optimize-form (nth 2 form) for-effect) | |
433 (byte-optimize-body (cdr (cdr (cdr form))) t))))) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
434 |
16276
f98b8c0b6414
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
14641
diff
changeset
|
435 ((memq fn '(save-excursion save-restriction save-current-buffer)) |
757 | 436 ;; those subrs which have an implicit progn; it's not quite good |
437 ;; enough to treat these like normal function calls. | |
438 ;; This can turn (save-excursion ...) into (save-excursion) which | |
439 ;; will be optimized away in the lap-optimize pass. | |
440 (cons fn (byte-optimize-body (cdr form) for-effect))) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
441 |
757 | 442 ((eq fn 'with-output-to-temp-buffer) |
443 ;; this is just like the above, except for the first argument. | |
444 (cons fn | |
445 (cons | |
446 (byte-optimize-form (nth 1 form) nil) | |
447 (byte-optimize-body (cdr (cdr form)) for-effect)))) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
448 |
757 | 449 ((eq fn 'if) |
36999 | 450 (when (< (length form) 3) |
39770 | 451 (byte-compile-warn "too few arguments for `if'")) |
757 | 452 (cons fn |
453 (cons (byte-optimize-form (nth 1 form) nil) | |
454 (cons | |
455 (byte-optimize-form (nth 2 form) for-effect) | |
456 (byte-optimize-body (nthcdr 3 form) for-effect))))) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
457 |
757 | 458 ((memq fn '(and or)) ; remember, and/or are control structures. |
459 ;; take forms off the back until we can't any more. | |
3591
507f64624555
Apply typo patches from Paul Eggert.
Jim Blandy <jimb@redhat.com>
parents:
3138
diff
changeset
|
460 ;; In the future it could conceivably be a problem that the |
757 | 461 ;; subexpressions of these forms are optimized in the reverse |
462 ;; order, but it's ok for now. | |
463 (if for-effect | |
464 (let ((backwards (reverse (cdr form)))) | |
465 (while (and backwards | |
466 (null (setcar backwards | |
467 (byte-optimize-form (car backwards) | |
468 for-effect)))) | |
469 (setq backwards (cdr backwards))) | |
470 (if (and (cdr form) (null backwards)) | |
471 (byte-compile-log | |
472 " all subforms of %s called for effect; deleted" form)) | |
473 (and backwards | |
48834
ffa5e816fcad
(byte-optimize-form-code-walker):
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
48823
diff
changeset
|
474 (cons fn (nreverse (mapcar 'byte-optimize-form backwards))))) |
757 | 475 (cons fn (mapcar 'byte-optimize-form (cdr form))))) |
476 | |
477 ((eq fn 'interactive) | |
39770 | 478 (byte-compile-warn "misplaced interactive spec: `%s'" |
757 | 479 (prin1-to-string form)) |
480 nil) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
481 |
757 | 482 ((memq fn '(defun defmacro function |
483 condition-case save-window-excursion)) | |
484 ;; These forms are compiled as constants or by breaking out | |
485 ;; all the subexpressions and compiling them separately. | |
486 form) | |
487 | |
488 ((eq fn 'unwind-protect) | |
489 ;; the "protected" part of an unwind-protect is compiled (and thus | |
490 ;; optimized) as a top-level form, so don't do it here. But the | |
491 ;; non-protected part has the same for-effect status as the | |
492 ;; unwind-protect itself. (The protected part is always for effect, | |
493 ;; but that isn't handled properly yet.) | |
494 (cons fn | |
495 (cons (byte-optimize-form (nth 1 form) for-effect) | |
496 (cdr (cdr form))))) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
497 |
757 | 498 ((eq fn 'catch) |
499 ;; the body of a catch is compiled (and thus optimized) as a | |
500 ;; top-level form, so don't do it here. The tag is never | |
501 ;; for-effect. The body should have the same for-effect status | |
502 ;; as the catch form itself, but that isn't handled properly yet. | |
503 (cons fn | |
504 (cons (byte-optimize-form (nth 1 form) nil) | |
505 (cdr (cdr form))))) | |
506 | |
47868
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
507 ((eq fn 'ignore) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
508 ;; Don't treat the args to `ignore' as being |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
509 ;; computed for effect. We want to avoid the warnings |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
510 ;; that might occur if they were treated that way. |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
511 ;; However, don't actually bother calling `ignore'. |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
512 `(prog1 nil . ,(mapcar 'byte-optimize-form (cdr form)))) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
513 |
757 | 514 ;; If optimization is on, this is the only place that macros are |
515 ;; expanded. If optimization is off, then macroexpansion happens | |
516 ;; in byte-compile-form. Otherwise, the macros are already expanded | |
517 ;; by the time that is reached. | |
518 ((not (eq form | |
519 (setq form (macroexpand form | |
520 byte-compile-macro-environment)))) | |
521 (byte-optimize-form form for-effect)) | |
20749
e87544dbfacb
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
20414
diff
changeset
|
522 |
e87544dbfacb
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
20414
diff
changeset
|
523 ;; Support compiler macros as in cl.el. |
e87544dbfacb
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
20414
diff
changeset
|
524 ((and (fboundp 'compiler-macroexpand) |
20874
223c220043af
(byte-optimize-form-code-walker): Only call compiler-macroexpand if
Richard M. Stallman <rms@gnu.org>
parents:
20780
diff
changeset
|
525 (symbolp (car-safe form)) |
223c220043af
(byte-optimize-form-code-walker): Only call compiler-macroexpand if
Richard M. Stallman <rms@gnu.org>
parents:
20780
diff
changeset
|
526 (get (car-safe form) 'cl-compiler-macro) |
20749
e87544dbfacb
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
20414
diff
changeset
|
527 (not (eq form |
58490
2176686c8a19
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
58210
diff
changeset
|
528 (with-no-warnings |
2176686c8a19
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
58210
diff
changeset
|
529 (setq form (compiler-macroexpand form)))))) |
20749
e87544dbfacb
(byte-optimize-form-code-walker):
Richard M. Stallman <rms@gnu.org>
parents:
20414
diff
changeset
|
530 (byte-optimize-form form for-effect)) |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
531 |
757 | 532 ((not (symbolp fn)) |
42267
76d12dafa83d
(byte-optimize-form-code-walker): Remove mocklisp case.
Pavel Janík <Pavel@Janik.cz>
parents:
42206
diff
changeset
|
533 (byte-compile-warn "`%s' is a malformed function" |
76d12dafa83d
(byte-optimize-form-code-walker): Remove mocklisp case.
Pavel Janík <Pavel@Janik.cz>
parents:
42206
diff
changeset
|
534 (prin1-to-string fn)) |
757 | 535 form) |
536 | |
537 ((and for-effect (setq tmp (get fn 'side-effect-free)) | |
538 (or byte-compile-delete-errors | |
539 (eq tmp 'error-free) | |
47868
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
540 ;; Detect the expansion of (pop foo). |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
541 ;; There is no need to compile the call to `car' there. |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
542 (and (eq fn 'car) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
543 (eq (car-safe (cadr form)) 'prog1) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
544 (let ((var (cadr (cadr form))) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
545 (last (nth 2 (cadr form)))) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
546 (and (symbolp var) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
547 (null (nthcdr 3 (cadr form))) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
548 (eq (car-safe last) 'setq) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
549 (eq (cadr last) var) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
550 (eq (car-safe (nth 2 last)) 'cdr) |
10e55e51ecdc
(byte-optimize-form-code-walker): Treat `ignore' specially--prevent
Richard M. Stallman <rms@gnu.org>
parents:
47317
diff
changeset
|
551 (eq (cadr (nth 2 last)) var)))) |
757 | 552 (progn |
76957
d1d2c4bdd413
(byte-optimize-form-code-walker): Print entire form.
Chong Yidong <cyd@stupidchicken.com>
parents:
75346
diff
changeset
|
553 (byte-compile-warn "value returned from %s is unused" |
d1d2c4bdd413
(byte-optimize-form-code-walker): Print entire form.
Chong Yidong <cyd@stupidchicken.com>
parents:
75346
diff
changeset
|
554 (prin1-to-string form)) |
757 | 555 nil))) |
556 (byte-compile-log " %s called for effect; deleted" fn) | |
557 ;; appending a nil here might not be necessary, but it can't hurt. | |
558 (byte-optimize-form | |
559 (cons 'progn (append (cdr form) '(nil))) t)) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
560 |
757 | 561 (t |
562 ;; Otherwise, no args can be considered to be for-effect, | |
563 ;; even if the called function is for-effect, because we | |
564 ;; don't know anything about that function. | |
77125
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
565 (let ((args (mapcar #'byte-optimize-form (cdr form)))) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
566 (if (and (get fn 'pure) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
567 (byte-optimize-all-constp args)) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
568 (list 'quote (apply fn (mapcar #'eval args))) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
569 (cons fn args))))))) |
757 | 570 |
77125
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
571 (defun byte-optimize-all-constp (list) |
78474
88c9f4e4160e
Replace `iff' in doc-strings and comments.
Glenn Morris <rgm@gnu.org>
parents:
78217
diff
changeset
|
572 "Non-nil if all elements of LIST satisfy `byte-compile-constp'." |
77125
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
573 (let ((constant t)) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
574 (while (and list constant) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
575 (unless (byte-compile-constp (car list)) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
576 (setq constant nil)) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
577 (setq list (cdr list))) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
578 constant)) |
757 | 579 |
580 (defun byte-optimize-form (form &optional for-effect) | |
581 "The source-level pass of the optimizer." | |
582 ;; | |
583 ;; First, optimize all sub-forms of this one. | |
584 (setq form (byte-optimize-form-code-walker form for-effect)) | |
585 ;; | |
586 ;; after optimizing all subforms, optimize this form until it doesn't | |
587 ;; optimize any further. This means that some forms will be passed through | |
588 ;; the optimizer many times, but that's necessary to make the for-effect | |
589 ;; processing do as much as possible. | |
590 ;; | |
591 (let (opt new) | |
592 (if (and (consp form) | |
593 (symbolp (car form)) | |
594 (or (and for-effect | |
595 ;; we don't have any of these yet, but we might. | |
596 (setq opt (get (car form) 'byte-for-effect-optimizer))) | |
597 (setq opt (get (car form) 'byte-optimizer))) | |
598 (not (eq form (setq new (funcall opt form))))) | |
599 (progn | |
600 ;; (if (equal form new) (error "bogus optimizer -- %s" opt)) | |
601 (byte-compile-log " %s\t==>\t%s" form new) | |
602 (setq new (byte-optimize-form new for-effect)) | |
603 new) | |
604 form))) | |
605 | |
606 | |
607 (defun byte-optimize-body (forms all-for-effect) | |
608 ;; optimize the cdr of a progn or implicit progn; all forms is a list of | |
609 ;; forms, all but the last of which are optimized with the assumption that | |
610 ;; they are being called for effect. the last is for-effect as well if | |
611 ;; all-for-effect is true. returns a new list of forms. | |
612 (let ((rest forms) | |
613 (result nil) | |
614 fe new) | |
615 (while rest | |
616 (setq fe (or all-for-effect (cdr rest))) | |
617 (setq new (and (car rest) (byte-optimize-form (car rest) fe))) | |
618 (if (or new (not fe)) | |
619 (setq result (cons new result))) | |
620 (setq rest (cdr rest))) | |
621 (nreverse result))) | |
622 | |
623 | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
624 ;; some source-level optimizers |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
625 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
626 ;; when writing optimizers, be VERY careful that the optimizer returns |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
627 ;; something not EQ to its argument if and ONLY if it has made a change. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
628 ;; This implies that you cannot simply destructively modify the list; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
629 ;; you must return something not EQ to it if you make an optimization. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
630 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
631 ;; It is now safe to optimize code such that it introduces new bindings. |
757 | 632 |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
633 (defsubst byte-compile-trueconstp (form) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
634 "Return non-nil if FORM always evaluates to a non-nil value." |
92450
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
635 (while (eq (car-safe form) 'progn) |
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
636 (setq form (car (last (cdr form))))) |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
637 (cond ((consp form) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
638 (case (car form) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
639 (quote (cadr form)) |
92450
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
640 ;; Can't use recursion in a defsubst. |
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
641 ;; (progn (byte-compile-trueconstp (car (last (cdr form))))) |
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
642 )) |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
643 ((not (symbolp form))) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
644 ((eq form t)) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
645 ((keywordp form)))) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
646 |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
647 (defsubst byte-compile-nilconstp (form) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
648 "Return non-nil if FORM always evaluates to a nil value." |
92450
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
649 (while (eq (car-safe form) 'progn) |
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
650 (setq form (car (last (cdr form))))) |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
651 (cond ((consp form) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
652 (case (car form) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
653 (quote (null (cadr form))) |
92450
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
654 ;; Can't use recursion in a defsubst. |
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
655 ;; (progn (byte-compile-nilconstp (car (last (cdr form))))) |
235b5ef31f41
(byte-compile-trueconstp, byte-compile-nilconstp): No recursion in defsubst.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
92426
diff
changeset
|
656 )) |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
657 ((not (symbolp form)) nil) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
658 ((null form)))) |
757 | 659 |
767
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
660 ;; If the function is being called with constant numeric args, |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
661 ;; evaluate as much as possible at compile-time. This optimizer |
767
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
662 ;; assumes that the function is associative, like + or *. |
757 | 663 (defun byte-optimize-associative-math (form) |
664 (let ((args nil) | |
665 (constants nil) | |
666 (rest (cdr form))) | |
667 (while rest | |
668 (if (numberp (car rest)) | |
669 (setq constants (cons (car rest) constants)) | |
670 (setq args (cons (car rest) args))) | |
671 (setq rest (cdr rest))) | |
672 (if (cdr constants) | |
673 (if args | |
674 (list (car form) | |
675 (apply (car form) constants) | |
676 (if (cdr args) | |
677 (cons (car form) (nreverse args)) | |
678 (car args))) | |
679 (apply (car form) constants)) | |
680 form))) | |
681 | |
767
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
682 ;; If the function is being called with constant numeric args, |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
683 ;; evaluate as much as possible at compile-time. This optimizer |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
684 ;; assumes that the function satisfies |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
685 ;; (op x1 x2 ... xn) == (op ...(op (op x1 x2) x3) ...xn) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
686 ;; like - and /. |
757 | 687 (defun byte-optimize-nonassociative-math (form) |
688 (if (or (not (numberp (car (cdr form)))) | |
689 (not (numberp (car (cdr (cdr form)))))) | |
690 form | |
691 (let ((constant (car (cdr form))) | |
692 (rest (cdr (cdr form)))) | |
693 (while (numberp (car rest)) | |
694 (setq constant (funcall (car form) constant (car rest)) | |
695 rest (cdr rest))) | |
696 (if rest | |
697 (cons (car form) (cons constant rest)) | |
698 constant)))) | |
699 | |
700 ;;(defun byte-optimize-associative-two-args-math (form) | |
701 ;; (setq form (byte-optimize-associative-math form)) | |
702 ;; (if (consp form) | |
703 ;; (byte-optimize-two-args-left form) | |
704 ;; form)) | |
705 | |
706 ;;(defun byte-optimize-nonassociative-two-args-math (form) | |
707 ;; (setq form (byte-optimize-nonassociative-math form)) | |
708 ;; (if (consp form) | |
709 ;; (byte-optimize-two-args-right form) | |
710 ;; form)) | |
711 | |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
712 (defun byte-optimize-approx-equal (x y) |
17680
fdb29fa454bf
(byte-optimize-approx-equal): Use <=, not <.
Richard M. Stallman <rms@gnu.org>
parents:
16941
diff
changeset
|
713 (<= (* (abs (- x y)) 100) (abs (+ x y)))) |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
714 |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
715 ;; Collect all the constants from FORM, after the STARTth arg, |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
716 ;; and apply FUN to them to make one argument at the end. |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
717 ;; For functions that can handle floats, that optimization |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
718 ;; can be incorrect because reordering can cause an overflow |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
719 ;; that would otherwise be avoided by encountering an arg that is a float. |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
720 ;; We avoid this problem by (1) not moving float constants and |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
721 ;; (2) not moving anything if it would cause an overflow. |
757 | 722 (defun byte-optimize-delay-constants-math (form start fun) |
723 ;; Merge all FORM's constants from number START, call FUN on them | |
724 ;; and put the result at the end. | |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
725 (let ((rest (nthcdr (1- start) form)) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
726 (orig form) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
727 ;; t means we must check for overflow. |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
728 (overflow (memq fun '(+ *)))) |
757 | 729 (while (cdr (setq rest (cdr rest))) |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
730 (if (integerp (car rest)) |
757 | 731 (let (constants) |
732 (setq form (copy-sequence form) | |
733 rest (nthcdr (1- start) form)) | |
734 (while (setq rest (cdr rest)) | |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
735 (cond ((integerp (car rest)) |
757 | 736 (setq constants (cons (car rest) constants)) |
737 (setcar rest nil)))) | |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
738 ;; If necessary, check now for overflow |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
739 ;; that might be caused by reordering. |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
740 (if (and overflow |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
741 ;; We have overflow if the result of doing the arithmetic |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
742 ;; on floats is not even close to the result |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
743 ;; of doing it on integers. |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
744 (not (byte-optimize-approx-equal |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
745 (apply fun (mapcar 'float constants)) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
746 (float (apply fun constants))))) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
747 (setq form orig) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
748 (setq form (nconc (delq nil form) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
749 (list (apply fun (nreverse constants))))))))) |
757 | 750 form)) |
751 | |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
752 (defsubst byte-compile-butlast (form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
753 (nreverse (cdr (reverse form)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
754 |
757 | 755 (defun byte-optimize-plus (form) |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
756 ;; Don't call `byte-optimize-delay-constants-math' (bug#1334). |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
757 ;;(setq form (byte-optimize-delay-constants-math form 1 '+)) |
757 | 758 (if (memq 0 form) (setq form (delq 0 (copy-sequence form)))) |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
759 ;; For (+ constants...), byte-optimize-predicate does the work. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
760 (when (memq nil (mapcar 'numberp (cdr form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
761 (cond |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
762 ;; (+ x 1) --> (1+ x) and (+ x -1) --> (1- x). |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
763 ((and (= (length form) 3) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
764 (or (memq (nth 1 form) '(1 -1)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
765 (memq (nth 2 form) '(1 -1)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
766 (let (integer other) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
767 (if (memq (nth 1 form) '(1 -1)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
768 (setq integer (nth 1 form) other (nth 2 form)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
769 (setq integer (nth 2 form) other (nth 1 form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
770 (setq form |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
771 (list (if (eq integer 1) '1+ '1-) other)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
772 ;; Here, we could also do |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
773 ;; (+ x y ... 1) --> (1+ (+ x y ...)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
774 ;; (+ x y ... -1) --> (1- (+ x y ...)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
775 ;; The resulting bytecode is smaller, but is it faster? -- cyd |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
776 )) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
777 (byte-optimize-predicate form)) |
757 | 778 |
779 (defun byte-optimize-minus (form) | |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
780 ;; Don't call `byte-optimize-delay-constants-math' (bug#1334). |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
781 ;;(setq form (byte-optimize-delay-constants-math form 2 '+)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
782 ;; Remove zeros. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
783 (when (and (nthcdr 3 form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
784 (memq 0 (cddr form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
785 (setq form (nconc (list (car form) (cadr form)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
786 (delq 0 (copy-sequence (cddr form))))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
787 ;; After the above, we must turn (- x) back into (- x 0) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
788 (or (cddr form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
789 (setq form (nconc form (list 0))))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
790 ;; For (- constants..), byte-optimize-predicate does the work. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
791 (when (memq nil (mapcar 'numberp (cdr form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
792 (cond |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
793 ;; (- x 1) --> (1- x) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
794 ((equal (nthcdr 2 form) '(1)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
795 (setq form (list '1- (nth 1 form)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
796 ;; (- x -1) --> (1+ x) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
797 ((equal (nthcdr 2 form) '(-1)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
798 (setq form (list '1+ (nth 1 form)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
799 ;; (- 0 x) --> (- x) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
800 ((and (eq (nth 1 form) 0) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
801 (= (length form) 3)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
802 (setq form (list '- (nth 2 form)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
803 ;; Here, we could also do |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
804 ;; (- x y ... 1) --> (1- (- x y ...)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
805 ;; (- x y ... -1) --> (1+ (- x y ...)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
806 ;; The resulting bytecode is smaller, but is it faster? -- cyd |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
807 )) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
808 (byte-optimize-predicate form)) |
757 | 809 |
810 (defun byte-optimize-multiply (form) | |
811 (setq form (byte-optimize-delay-constants-math form 1 '*)) | |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
812 ;; For (* constants..), byte-optimize-predicate does the work. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
813 (when (memq nil (mapcar 'numberp (cdr form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
814 ;; After `byte-optimize-predicate', if there is a INTEGER constant |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
815 ;; in FORM, it is in the last element. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
816 (let ((last (car (reverse (cdr form))))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
817 (cond |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
818 ;; Would handling (* ... 0) here cause floating point errors? |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
819 ;; See bug#1334. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
820 ((eq 1 last) (setq form (byte-compile-butlast form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
821 ((eq -1 last) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
822 (setq form (list '- (if (nthcdr 3 form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
823 (byte-compile-butlast form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
824 (nth 1 form)))))))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
825 (byte-optimize-predicate form)) |
757 | 826 |
827 (defun byte-optimize-divide (form) | |
828 (setq form (byte-optimize-delay-constants-math form 2 '*)) | |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
829 ;; After `byte-optimize-predicate', if there is a INTEGER constant |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
830 ;; in FORM, it is in the last element. |
757 | 831 (let ((last (car (reverse (cdr (cdr form)))))) |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
832 (cond |
99772
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
833 ;; Runtime error (leave it intact). |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
834 ((or (null last) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
835 (eq last 0) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
836 (memql 0.0 (cddr form)))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
837 ;; No constants in expression |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
838 ((not (numberp last))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
839 ;; For (* constants..), byte-optimize-predicate does the work. |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
840 ((null (memq nil (mapcar 'numberp (cdr form))))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
841 ;; (/ x y.. 1) --> (/ x y..) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
842 ((and (eq last 1) (nthcdr 3 form)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
843 (setq form (byte-compile-butlast form))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
844 ;; (/ x -1), (/ x .. -1) --> (- x), (- (/ x ..)) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
845 ((eq last -1) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
846 (setq form (list '- (if (nthcdr 3 form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
847 (byte-compile-butlast form) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
848 (nth 1 form))))))) |
57a488466504
(byte-compile-butlast): Move up in file.
Chong Yidong <cyd@stupidchicken.com>
parents:
94655
diff
changeset
|
849 (byte-optimize-predicate form)) |
757 | 850 |
851 (defun byte-optimize-logmumble (form) | |
852 (setq form (byte-optimize-delay-constants-math form 1 (car form))) | |
853 (byte-optimize-predicate | |
854 (cond ((memq 0 form) | |
855 (setq form (if (eq (car form) 'logand) | |
856 (cons 'progn (cdr form)) | |
857 (delq 0 (copy-sequence form))))) | |
858 ((and (eq (car-safe form) 'logior) | |
859 (memq -1 form)) | |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
860 (cons 'progn (cdr form))) |
757 | 861 (form)))) |
862 | |
863 | |
864 (defun byte-optimize-binary-predicate (form) | |
865 (if (byte-compile-constp (nth 1 form)) | |
866 (if (byte-compile-constp (nth 2 form)) | |
867 (condition-case () | |
868 (list 'quote (eval form)) | |
869 (error form)) | |
870 ;; This can enable some lapcode optimizations. | |
871 (list (car form) (nth 2 form) (nth 1 form))) | |
872 form)) | |
873 | |
874 (defun byte-optimize-predicate (form) | |
875 (let ((ok t) | |
876 (rest (cdr form))) | |
877 (while (and rest ok) | |
878 (setq ok (byte-compile-constp (car rest)) | |
879 rest (cdr rest))) | |
880 (if ok | |
881 (condition-case () | |
882 (list 'quote (eval form)) | |
883 (error form)) | |
884 form))) | |
885 | |
886 (defun byte-optimize-identity (form) | |
887 (if (and (cdr form) (null (cdr (cdr form)))) | |
888 (nth 1 form) | |
39770 | 889 (byte-compile-warn "identity called with %d arg%s, but requires 1" |
757 | 890 (length (cdr form)) |
891 (if (= 1 (length (cdr form))) "" "s")) | |
892 form)) | |
893 | |
894 (put 'identity 'byte-optimizer 'byte-optimize-identity) | |
895 | |
896 (put '+ 'byte-optimizer 'byte-optimize-plus) | |
897 (put '* 'byte-optimizer 'byte-optimize-multiply) | |
898 (put '- 'byte-optimizer 'byte-optimize-minus) | |
899 (put '/ 'byte-optimizer 'byte-optimize-divide) | |
900 (put 'max 'byte-optimizer 'byte-optimize-associative-math) | |
901 (put 'min 'byte-optimizer 'byte-optimize-associative-math) | |
902 | |
903 (put '= 'byte-optimizer 'byte-optimize-binary-predicate) | |
904 (put 'eq 'byte-optimizer 'byte-optimize-binary-predicate) | |
905 (put 'equal 'byte-optimizer 'byte-optimize-binary-predicate) | |
906 (put 'string= 'byte-optimizer 'byte-optimize-binary-predicate) | |
907 (put 'string-equal 'byte-optimizer 'byte-optimize-binary-predicate) | |
908 | |
909 (put '< 'byte-optimizer 'byte-optimize-predicate) | |
910 (put '> 'byte-optimizer 'byte-optimize-predicate) | |
911 (put '<= 'byte-optimizer 'byte-optimize-predicate) | |
912 (put '>= 'byte-optimizer 'byte-optimize-predicate) | |
913 (put '1+ 'byte-optimizer 'byte-optimize-predicate) | |
914 (put '1- 'byte-optimizer 'byte-optimize-predicate) | |
915 (put 'not 'byte-optimizer 'byte-optimize-predicate) | |
916 (put 'null 'byte-optimizer 'byte-optimize-predicate) | |
917 (put 'memq 'byte-optimizer 'byte-optimize-predicate) | |
918 (put 'consp 'byte-optimizer 'byte-optimize-predicate) | |
919 (put 'listp 'byte-optimizer 'byte-optimize-predicate) | |
920 (put 'symbolp 'byte-optimizer 'byte-optimize-predicate) | |
921 (put 'stringp 'byte-optimizer 'byte-optimize-predicate) | |
922 (put 'string< 'byte-optimizer 'byte-optimize-predicate) | |
923 (put 'string-lessp 'byte-optimizer 'byte-optimize-predicate) | |
924 | |
925 (put 'logand 'byte-optimizer 'byte-optimize-logmumble) | |
926 (put 'logior 'byte-optimizer 'byte-optimize-logmumble) | |
927 (put 'logxor 'byte-optimizer 'byte-optimize-logmumble) | |
928 (put 'lognot 'byte-optimizer 'byte-optimize-predicate) | |
929 | |
930 (put 'car 'byte-optimizer 'byte-optimize-predicate) | |
931 (put 'cdr 'byte-optimizer 'byte-optimize-predicate) | |
932 (put 'car-safe 'byte-optimizer 'byte-optimize-predicate) | |
933 (put 'cdr-safe 'byte-optimizer 'byte-optimize-predicate) | |
934 | |
935 | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
936 ;; I'm not convinced that this is necessary. Doesn't the optimizer loop |
757 | 937 ;; take care of this? - Jamie |
938 ;; I think this may some times be necessary to reduce ie (quote 5) to 5, | |
3591
507f64624555
Apply typo patches from Paul Eggert.
Jim Blandy <jimb@redhat.com>
parents:
3138
diff
changeset
|
939 ;; so arithmetic optimizers recognize the numeric constant. - Hallvard |
757 | 940 (put 'quote 'byte-optimizer 'byte-optimize-quote) |
941 (defun byte-optimize-quote (form) | |
942 (if (or (consp (nth 1 form)) | |
943 (and (symbolp (nth 1 form)) | |
27823 | 944 (not (byte-compile-const-symbol-p form)))) |
757 | 945 form |
946 (nth 1 form))) | |
947 | |
948 (defun byte-optimize-zerop (form) | |
949 (cond ((numberp (nth 1 form)) | |
950 (eval form)) | |
951 (byte-compile-delete-errors | |
952 (list '= (nth 1 form) 0)) | |
953 (form))) | |
954 | |
955 (put 'zerop 'byte-optimizer 'byte-optimize-zerop) | |
956 | |
957 (defun byte-optimize-and (form) | |
958 ;; Simplify if less than 2 args. | |
959 ;; if there is a literal nil in the args to `and', throw it and following | |
960 ;; forms away, and surround the `and' with (progn ... nil). | |
961 (cond ((null (cdr form))) | |
962 ((memq nil form) | |
963 (list 'progn | |
964 (byte-optimize-and | |
965 (prog1 (setq form (copy-sequence form)) | |
966 (while (nth 1 form) | |
967 (setq form (cdr form))) | |
968 (setcdr form nil))) | |
969 nil)) | |
970 ((null (cdr (cdr form))) | |
971 (nth 1 form)) | |
972 ((byte-optimize-predicate form)))) | |
973 | |
974 (defun byte-optimize-or (form) | |
975 ;; Throw away nil's, and simplify if less than 2 args. | |
976 ;; If there is a literal non-nil constant in the args to `or', throw away all | |
977 ;; following forms. | |
978 (if (memq nil form) | |
979 (setq form (delq nil (copy-sequence form)))) | |
980 (let ((rest form)) | |
981 (while (cdr (setq rest (cdr rest))) | |
982 (if (byte-compile-trueconstp (car rest)) | |
983 (setq form (copy-sequence form) | |
984 rest (setcdr (memq (car rest) form) nil)))) | |
985 (if (cdr (cdr form)) | |
986 (byte-optimize-predicate form) | |
987 (nth 1 form)))) | |
988 | |
989 (defun byte-optimize-cond (form) | |
990 ;; if any clauses have a literal nil as their test, throw them away. | |
991 ;; if any clause has a literal non-nil constant as its test, throw | |
992 ;; away all following clauses. | |
993 (let (rest) | |
994 ;; This must be first, to reduce (cond (t ...) (nil)) to (progn t ...) | |
995 (while (setq rest (assq nil (cdr form))) | |
996 (setq form (delq rest (copy-sequence form)))) | |
997 (if (memq nil (cdr form)) | |
998 (setq form (delq nil (copy-sequence form)))) | |
999 (setq rest form) | |
1000 (while (setq rest (cdr rest)) | |
1001 (cond ((byte-compile-trueconstp (car-safe (car rest))) | |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1002 ;; This branch will always be taken: kill the subsequent ones. |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1003 (cond ((eq rest (cdr form)) ;First branch of `cond'. |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1004 (setq form `(progn ,@(car rest)))) |
757 | 1005 ((cdr rest) |
1006 (setq form (copy-sequence form)) | |
1007 (setcdr (memq (car rest) form) nil))) | |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1008 (setq rest nil)) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1009 ((and (consp (car rest)) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1010 (byte-compile-nilconstp (caar rest))) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1011 ;; This branch will never be taken: kill its body. |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1012 (setcdr (car rest) nil))))) |
757 | 1013 ;; |
1014 ;; Turn (cond (( <x> )) ... ) into (or <x> (cond ... )) | |
1015 (if (eq 'cond (car-safe form)) | |
1016 (let ((clauses (cdr form))) | |
1017 (if (and (consp (car clauses)) | |
1018 (null (cdr (car clauses)))) | |
1019 (list 'or (car (car clauses)) | |
1020 (byte-optimize-cond | |
1021 (cons (car form) (cdr (cdr form))))) | |
1022 form)) | |
1023 form)) | |
1024 | |
1025 (defun byte-optimize-if (form) | |
82790
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1026 ;; (if (progn <insts> <test>) <rest>) ==> (progn <insts> (if <test> <rest>)) |
757 | 1027 ;; (if <true-constant> <then> <else...>) ==> <then> |
1028 ;; (if <false-constant> <then> <else...>) ==> (progn <else...>) | |
1029 ;; (if <test> nil <else...>) ==> (if (not <test>) (progn <else...>)) | |
1030 ;; (if <test> <then> nil) ==> (if <test> <then>) | |
1031 (let ((clause (nth 1 form))) | |
82822
4d04d9500eaf
(byte-optimize-if): Don't presume `clause' is a list.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82790
diff
changeset
|
1032 (cond ((and (eq (car-safe clause) 'progn) |
4d04d9500eaf
(byte-optimize-if): Don't presume `clause' is a list.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82790
diff
changeset
|
1033 ;; `clause' is a proper list. |
4d04d9500eaf
(byte-optimize-if): Don't presume `clause' is a list.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82790
diff
changeset
|
1034 (null (cdr (last clause)))) |
82790
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1035 (if (null (cddr clause)) |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1036 ;; A trivial `progn'. |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1037 (byte-optimize-if `(if ,(cadr clause) ,@(nthcdr 2 form))) |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1038 (nconc (butlast clause) |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1039 (list |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1040 (byte-optimize-if |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1041 `(if ,(car (last clause)) ,@(nthcdr 2 form))))))) |
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1042 ((byte-compile-trueconstp clause) |
86056
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1043 `(progn ,clause ,(nth 2 form))) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1044 ((byte-compile-nilconstp clause) |
e0931ee6cc83
* emacs-lisp/byte-opt.el (byte-compile-trueconstp): Handle more
Dan Nicolaescu <dann@ics.uci.edu>
parents:
86003
diff
changeset
|
1045 `(progn ,clause ,@(nthcdr 3 form))) |
757 | 1046 ((nth 2 form) |
1047 (if (equal '(nil) (nthcdr 3 form)) | |
1048 (list 'if clause (nth 2 form)) | |
1049 form)) | |
1050 ((or (nth 3 form) (nthcdr 4 form)) | |
12550
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1051 (list 'if |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1052 ;; Don't make a double negative; |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1053 ;; instead, take away the one that is there. |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1054 (if (and (consp clause) (memq (car clause) '(not null)) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1055 (= (length clause) 2)) ; (not xxxx) or (not (xxxx)) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1056 (nth 1 clause) |
c33dd1c62d72
(byte-optimize-nth, byte-optimize-nthcdr):
Karl Heuer <kwzh@gnu.org>
parents:
11509
diff
changeset
|
1057 (list 'not clause)) |
757 | 1058 (if (nthcdr 4 form) |
1059 (cons 'progn (nthcdr 3 form)) | |
1060 (nth 3 form)))) | |
1061 (t | |
1062 (list 'progn clause nil))))) | |
1063 | |
1064 (defun byte-optimize-while (form) | |
36999 | 1065 (when (< (length form) 2) |
39770 | 1066 (byte-compile-warn "too few arguments for `while'")) |
757 | 1067 (if (nth 1 form) |
1068 form)) | |
1069 | |
1070 (put 'and 'byte-optimizer 'byte-optimize-and) | |
1071 (put 'or 'byte-optimizer 'byte-optimize-or) | |
1072 (put 'cond 'byte-optimizer 'byte-optimize-cond) | |
1073 (put 'if 'byte-optimizer 'byte-optimize-if) | |
1074 (put 'while 'byte-optimizer 'byte-optimize-while) | |
1075 | |
1076 ;; byte-compile-negation-optimizer lives in bytecomp.el | |
1077 (put '/= 'byte-optimizer 'byte-compile-negation-optimizer) | |
1078 (put 'atom 'byte-optimizer 'byte-compile-negation-optimizer) | |
1079 (put 'nlistp 'byte-optimizer 'byte-compile-negation-optimizer) | |
1080 | |
1081 | |
1082 (defun byte-optimize-funcall (form) | |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
1083 ;; (funcall (lambda ...) ...) ==> ((lambda ...) ...) |
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
1084 ;; (funcall foo ...) ==> (foo ...) |
757 | 1085 (let ((fn (nth 1 form))) |
1086 (if (memq (car-safe fn) '(quote function)) | |
1087 (cons (nth 1 fn) (cdr (cdr form))) | |
1088 form))) | |
1089 | |
1090 (defun byte-optimize-apply (form) | |
1091 ;; If the last arg is a literal constant, turn this into a funcall. | |
1092 ;; The funcall optimizer can then transform (funcall 'foo ...) -> (foo ...). | |
1093 (let ((fn (nth 1 form)) | |
1094 (last (nth (1- (length form)) form))) ; I think this really is fastest | |
1095 (or (if (or (null last) | |
1096 (eq (car-safe last) 'quote)) | |
1097 (if (listp (nth 1 last)) | |
1098 (let ((butlast (nreverse (cdr (reverse (cdr (cdr form))))))) | |
957 | 1099 (nconc (list 'funcall fn) butlast |
29580
2f88e6f0d32b
(byte-compile-log-lap-1)
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
29053
diff
changeset
|
1100 (mapcar (lambda (x) (list 'quote x)) (nth 1 last)))) |
757 | 1101 (byte-compile-warn |
39770 | 1102 "last arg to apply can't be a literal atom: `%s'" |
757 | 1103 (prin1-to-string last)) |
1104 nil)) | |
1105 form))) | |
1106 | |
1107 (put 'funcall 'byte-optimizer 'byte-optimize-funcall) | |
1108 (put 'apply 'byte-optimizer 'byte-optimize-apply) | |
1109 | |
1110 | |
1111 (put 'let 'byte-optimizer 'byte-optimize-letX) | |
1112 (put 'let* 'byte-optimizer 'byte-optimize-letX) | |
1113 (defun byte-optimize-letX (form) | |
1114 (cond ((null (nth 1 form)) | |
1115 ;; No bindings | |
1116 (cons 'progn (cdr (cdr form)))) | |
1117 ((or (nth 2 form) (nthcdr 3 form)) | |
1118 form) | |
1119 ;; The body is nil | |
1120 ((eq (car form) 'let) | |
11509
853f52a85d11
(byte-optimize-letX): Use car-safe and cdr-safe.
Richard M. Stallman <rms@gnu.org>
parents:
11203
diff
changeset
|
1121 (append '(progn) (mapcar 'car-safe (mapcar 'cdr-safe (nth 1 form))) |
853f52a85d11
(byte-optimize-letX): Use car-safe and cdr-safe.
Richard M. Stallman <rms@gnu.org>
parents:
11203
diff
changeset
|
1122 '(nil))) |
757 | 1123 (t |
1124 (let ((binds (reverse (nth 1 form)))) | |
1125 (list 'let* (reverse (cdr binds)) (nth 1 (car binds)) nil))))) | |
1126 | |
1127 | |
1128 (put 'nth 'byte-optimizer 'byte-optimize-nth) | |
1129 (defun byte-optimize-nth (form) | |
49041 | 1130 (if (= (safe-length form) 3) |
1131 (if (memq (nth 1 form) '(0 1)) | |
1132 (list 'car (if (zerop (nth 1 form)) | |
1133 (nth 2 form) | |
1134 (list 'cdr (nth 2 form)))) | |
1135 (byte-optimize-predicate form)) | |
1136 form)) | |
757 | 1137 |
1138 (put 'nthcdr 'byte-optimizer 'byte-optimize-nthcdr) | |
1139 (defun byte-optimize-nthcdr (form) | |
49041 | 1140 (if (= (safe-length form) 3) |
1141 (if (memq (nth 1 form) '(0 1 2)) | |
1142 (let ((count (nth 1 form))) | |
1143 (setq form (nth 2 form)) | |
1144 (while (>= (setq count (1- count)) 0) | |
1145 (setq form (list 'cdr form))) | |
1146 form) | |
1147 (byte-optimize-predicate form)) | |
1148 form)) | |
20210
622d0b6c6445
(byte-optimize-concat): New function.
Karl Heuer <kwzh@gnu.org>
parents:
17680
diff
changeset
|
1149 |
48823 | 1150 ;; Fixme: delete-char -> delete-region (byte-coded) |
1151 ;; optimize string-as-unibyte, string-as-multibyte, string-make-unibyte, | |
1152 ;; string-make-multibyte for constant args. | |
1153 | |
1154 (put 'featurep 'byte-optimizer 'byte-optimize-featurep) | |
1155 (defun byte-optimize-featurep (form) | |
78627
e8ce4af3f42d
(byte-optimize-featurep): Also handle `sxemacs'.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
78474
diff
changeset
|
1156 ;; Emacs-21's byte-code doesn't run under XEmacs or SXEmacs anyway, so we |
e8ce4af3f42d
(byte-optimize-featurep): Also handle `sxemacs'.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
78474
diff
changeset
|
1157 ;; can safely optimize away this test. |
85513
dfdbd485bfa6
(byte-optimize-featurep): Fix paren typo.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
85183
diff
changeset
|
1158 (if (member (cdr-safe form) '(((quote xemacs)) ((quote sxemacs)))) |
48823 | 1159 nil |
86003
e33327200372
* emacs-lisp/byte-opt.el (byte-optimize-featurep): Optimize
Dan Nicolaescu <dann@ics.uci.edu>
parents:
85513
diff
changeset
|
1160 (if (member (cdr-safe form) '(((quote emacs)))) |
e33327200372
* emacs-lisp/byte-opt.el (byte-optimize-featurep): Optimize
Dan Nicolaescu <dann@ics.uci.edu>
parents:
85513
diff
changeset
|
1161 t |
e33327200372
* emacs-lisp/byte-opt.el (byte-optimize-featurep): Optimize
Dan Nicolaescu <dann@ics.uci.edu>
parents:
85513
diff
changeset
|
1162 form))) |
50306
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1163 |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1164 (put 'set 'byte-optimizer 'byte-optimize-set) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1165 (defun byte-optimize-set (form) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1166 (let ((var (car-safe (cdr-safe form)))) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1167 (cond |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1168 ((and (eq (car-safe var) 'quote) (consp (cdr var))) |
50307
78b51ac98da0
(byte-optimize-set): Avoid CLism.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
50306
diff
changeset
|
1169 `(setq ,(cadr var) ,@(cddr form))) |
50306
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1170 ((and (eq (car-safe var) 'make-local-variable) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1171 (eq (car-safe (setq var (car-safe (cdr var)))) 'quote) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1172 (consp (cdr var))) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1173 `(progn ,(cadr form) (setq ,(cadr var) ,@(cddr form)))) |
2f19a8ed285b
(byte-optimize-set): New. Turn `set' into `setq' when applicable.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
49598
diff
changeset
|
1174 (t form)))) |
757 | 1175 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1176 ;; enumerating those functions which need not be called if the returned |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1177 ;; value is not used. That is, something like |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1178 ;; (progn (list (something-with-side-effects) (yow)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1179 ;; (foo)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1180 ;; may safely be turned into |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1181 ;; (progn (progn (something-with-side-effects) (yow)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1182 ;; (foo)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1183 ;; Further optimizations will turn (progn (list 1 2 3) 'foo) into 'foo. |
757 | 1184 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1185 ;; Some of these functions have the side effect of allocating memory |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1186 ;; and it would be incorrect to replace two calls with one. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1187 ;; But we don't try to do those kinds of optimizations, |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1188 ;; so it is safe to list such functions here. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1189 ;; Some of these functions return values that depend on environment |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1190 ;; state, so that constant folding them would be wrong, |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1191 ;; but we don't do constant folding based on this list. |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1192 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1193 ;; However, at present the only optimization we normally do |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1194 ;; is delete calls that need not occur, and we only do that |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1195 ;; with the error-free functions. |
47317 | 1196 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1197 ;; I wonder if I missed any :-\) |
757 | 1198 (let ((side-effect-free-fns |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1199 '(% * + - / /= 1+ 1- < <= = > >= abs acos append aref ash asin atan |
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1200 assoc assq |
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1201 boundp buffer-file-name buffer-local-variables buffer-modified-p |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1202 buffer-substring byte-code-function-p |
29053
565418f2e425
Update side-effect free function lists.
Dave Love <fx@gnu.org>
parents:
28440
diff
changeset
|
1203 capitalize car-less-than-car car cdr ceiling char-after char-before |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1204 char-equal char-to-string char-width |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1205 compare-strings concat coordinates-in-window-p |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1206 copy-alist copy-sequence copy-marker cos count-lines |
93209
a9665f39ec60
(side-effect-free-fns): Fix typo.
Juanma Barranquero <lekktu@gmail.com>
parents:
92450
diff
changeset
|
1207 decode-char |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1208 decode-time default-boundp default-value documentation downcase |
89483 | 1209 elt encode-char exp expt encode-time error-message-string |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1210 fboundp fceiling featurep ffloor |
757 | 1211 file-directory-p file-exists-p file-locked-p file-name-absolute-p |
1212 file-newer-than-file-p file-readable-p file-symlink-p file-writable-p | |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1213 float float-time floor format format-time-string frame-visible-p |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1214 fround ftruncate |
26941
8584ef89a2bd
Don't put optimization info on `eql'.
Dave Love <fx@gnu.org>
parents:
25621
diff
changeset
|
1215 get gethash get-buffer get-buffer-window getenv get-file-buffer |
8584ef89a2bd
Don't put optimization info on `eql'.
Dave Love <fx@gnu.org>
parents:
25621
diff
changeset
|
1216 hash-table-count |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1217 int-to-string intern-soft |
25621
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1218 keymap-parent |
27823 | 1219 length local-variable-if-set-p local-variable-p log log10 logand |
89342
8cf1af9a4468
(side-effect-free-fns): Add langinfo,
Dave Love <fx@gnu.org>
parents:
89205
diff
changeset
|
1220 logb logior lognot logxor lsh langinfo |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1221 make-list make-string make-symbol |
48467 | 1222 marker-buffer max member memq min mod multibyte-char-to-unibyte |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1223 next-window nth nthcdr number-to-string |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1224 parse-colon-path plist-get plist-member |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1225 prefix-numeric-value previous-window prin1-to-string propertize |
104914
1a8afcc5fb20
(degrees-to-radians): Mark as free from side effects.
Glenn Morris <rgm@gnu.org>
parents:
100908
diff
changeset
|
1226 degrees-to-radians |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1227 radians-to-degrees rassq rassoc read-from-string regexp-quote |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1228 region-beginning region-end reverse round |
29053
565418f2e425
Update side-effect free function lists.
Dave Love <fx@gnu.org>
parents:
28440
diff
changeset
|
1229 sin sqrt string string< string= string-equal string-lessp string-to-char |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1230 string-to-int string-to-number substring sxhash symbol-function |
48467 | 1231 symbol-name symbol-plist symbol-value string-make-unibyte |
1232 string-make-multibyte string-as-multibyte string-as-unibyte | |
89483 | 1233 string-to-multibyte |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1234 tan truncate |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1235 unibyte-char-to-multibyte upcase user-full-name |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1236 user-login-name user-original-login-name user-variable-p |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1237 vconcat |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1238 window-buffer window-dedicated-p window-edges window-height |
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1239 window-hscroll window-minibuffer-p window-width |
757 | 1240 zerop)) |
1241 (side-effect-and-error-free-fns | |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1242 '(arrayp atom |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
1243 bobp bolp bool-vector-p |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1244 buffer-end buffer-list buffer-size buffer-string bufferp |
88611 | 1245 car-safe case-table-p cdr-safe char-or-string-p characterp |
88804
c97005389b75
(side-effect-free-fns): Add decode-char,
Dave Love <fx@gnu.org>
parents:
88611
diff
changeset
|
1246 charsetp commandp cons consp |
25621
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1247 current-buffer current-global-map current-indentation |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1248 current-local-map current-minor-mode-maps current-time |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1249 current-time-string current-time-zone |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1250 eobp eolp eq equal eventp |
29053
565418f2e425
Update side-effect free function lists.
Dave Love <fx@gnu.org>
parents:
28440
diff
changeset
|
1251 floatp following-char framep |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1252 get-largest-window get-lru-window |
26941
8584ef89a2bd
Don't put optimization info on `eql'.
Dave Love <fx@gnu.org>
parents:
25621
diff
changeset
|
1253 hash-table-p |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1254 identity ignore integerp integer-or-marker-p interactive-p |
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1255 invocation-directory invocation-name |
25621
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1256 keymapp |
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1257 line-beginning-position line-end-position list listp |
89483 | 1258 make-marker mark mark-marker markerp max-char |
1259 memory-limit minibuffer-window | |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1260 mouse-movement-p |
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1261 natnump nlistp not null number-or-marker-p numberp |
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1262 one-window-p overlayp |
88804
c97005389b75
(side-effect-free-fns): Add decode-char,
Dave Love <fx@gnu.org>
parents:
88611
diff
changeset
|
1263 point point-marker point-min point-max preceding-char primary-charset |
c97005389b75
(side-effect-free-fns): Add decode-char,
Dave Love <fx@gnu.org>
parents:
88611
diff
changeset
|
1264 processp |
25621
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1265 recent-keys recursion-depth |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1266 safe-length selected-frame selected-window sequencep |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1267 standard-case-table standard-syntax-table stringp subrp symbolp |
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1268 syntax-table syntax-table-p |
25621
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1269 this-command-keys this-command-keys-vector this-single-command-keys |
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1270 this-single-command-raw-keys |
5315
55a8d59088c1
Add side-effect-free props for many functions.
Richard M. Stallman <rms@gnu.org>
parents:
4755
diff
changeset
|
1271 user-real-login-name user-real-uid user-uid |
25621
e263170c30d0
(byte-optimize-backward-char, byte-optimize-backward-word): New
Dave Love <fx@gnu.org>
parents:
25557
diff
changeset
|
1272 vector vectorp visible-frame-list |
44271
8788778fec51
(side-effect-free-fns, side-effect-and-error-free-fns):
Richard M. Stallman <rms@gnu.org>
parents:
42267
diff
changeset
|
1273 wholenump window-configuration-p window-live-p windowp))) |
757 | 1274 (while side-effect-free-fns |
1275 (put (car side-effect-free-fns) 'side-effect-free t) | |
1276 (setq side-effect-free-fns (cdr side-effect-free-fns))) | |
1277 (while side-effect-and-error-free-fns | |
1278 (put (car side-effect-and-error-free-fns) 'side-effect-free 'error-free) | |
1279 (setq side-effect-and-error-free-fns (cdr side-effect-and-error-free-fns))) | |
1280 nil) | |
1281 | |
77125
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1282 |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1283 ;; pure functions are side-effect free functions whose values depend |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1284 ;; only on their arguments. For these functions, calls with constant |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1285 ;; arguments can be evaluated at compile time. This may shift run time |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1286 ;; errors to compile time. |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1287 |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1288 (let ((pure-fns |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1289 '(concat symbol-name regexp-opt regexp-quote string-to-syntax))) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1290 (while pure-fns |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1291 (put (car pure-fns) 'pure t) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1292 (setq pure-fns (cdr pure-fns))) |
d09092672432
(byte-optimize-form-code-walker): Evaluate pure function calls if possible.
Chong Yidong <cyd@stupidchicken.com>
parents:
77118
diff
changeset
|
1293 nil) |
757 | 1294 |
1295 (defun byte-compile-splice-in-already-compiled-code (form) | |
1296 ;; form is (byte-code "..." [...] n) | |
1297 (if (not (memq byte-optimize '(t lap))) | |
1298 (byte-compile-normal-call form) | |
1299 (byte-inline-lapcode | |
1300 (byte-decompile-bytecode-1 (nth 1 form) (nth 2 form) t)) | |
1301 (setq byte-compile-maxdepth (max (+ byte-compile-depth (nth 3 form)) | |
1302 byte-compile-maxdepth)) | |
1303 (setq byte-compile-depth (1+ byte-compile-depth)))) | |
1304 | |
1305 (put 'byte-code 'byte-compile 'byte-compile-splice-in-already-compiled-code) | |
1306 | |
1307 | |
1308 (defconst byte-constref-ops | |
1309 '(byte-constant byte-constant2 byte-varref byte-varset byte-varbind)) | |
1310 | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1311 ;; This function extracts the bitfields from variable-length opcodes. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1312 ;; Originally defined in disass.el (which no longer uses it.) |
757 | 1313 |
1314 (defun disassemble-offset () | |
1315 "Don't call this!" | |
1316 ;; fetch and return the offset for the current opcode. | |
42206 | 1317 ;; return nil if this opcode has no offset |
757 | 1318 ;; OP, PTR and BYTES are used and set dynamically |
1319 (defvar op) | |
1320 (defvar ptr) | |
1321 (defvar bytes) | |
1322 (cond ((< op byte-nth) | |
1323 (let ((tem (logand op 7))) | |
1324 (setq op (logand op 248)) | |
1325 (cond ((eq tem 6) | |
1326 (setq ptr (1+ ptr)) ;offset in next byte | |
1327 (aref bytes ptr)) | |
1328 ((eq tem 7) | |
1329 (setq ptr (1+ ptr)) ;offset in next 2 bytes | |
1330 (+ (aref bytes ptr) | |
1331 (progn (setq ptr (1+ ptr)) | |
1332 (lsh (aref bytes ptr) 8)))) | |
1333 (t tem)))) ;offset was in opcode | |
1334 ((>= op byte-constant) | |
1335 (prog1 (- op byte-constant) ;offset in opcode | |
1336 (setq op byte-constant))) | |
1337 ((and (>= op byte-constant2) | |
1338 (<= op byte-goto-if-not-nil-else-pop)) | |
1339 (setq ptr (1+ ptr)) ;offset in next 2 bytes | |
1340 (+ (aref bytes ptr) | |
1341 (progn (setq ptr (1+ ptr)) | |
1342 (lsh (aref bytes ptr) 8)))) | |
848 | 1343 ((and (>= op byte-listN) |
757 | 1344 (<= op byte-insertN)) |
1345 (setq ptr (1+ ptr)) ;offset in next byte | |
1346 (aref bytes ptr)))) | |
1347 | |
1348 | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1349 ;; This de-compiler is used for inline expansion of compiled functions, |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1350 ;; and by the disassembler. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1351 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1352 ;; This list contains numbers, which are pc values, |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1353 ;; before each instruction. |
757 | 1354 (defun byte-decompile-bytecode (bytes constvec) |
82790
346e6c150fa0
(byte-optimize-if): Move `progn' out of the test
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
82365
diff
changeset
|
1355 "Turn BYTECODE into lapcode, referring to CONSTVEC." |
757 | 1356 (let ((byte-compile-constants nil) |
1357 (byte-compile-variables nil) | |
1358 (byte-compile-tag-number 0)) | |
1359 (byte-decompile-bytecode-1 bytes constvec))) | |
1360 | |
767
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
1361 ;; As byte-decompile-bytecode, but updates |
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
1362 ;; byte-compile-{constants, variables, tag-number}. |
8294
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1363 ;; If MAKE-SPLICEABLE is true, then `return' opcodes are replaced |
767
02bfc9709961
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
757
diff
changeset
|
1364 ;; with `goto's destined for the end of the code. |
8294
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1365 ;; That is for use by the compiler. |
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1366 ;; If MAKE-SPLICEABLE is nil, we are being called for the disassembler. |
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1367 ;; In that case, we put a pc value into the list |
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1368 ;; before each insn (or its label). |
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1369 (defun byte-decompile-bytecode-1 (bytes constvec &optional make-spliceable) |
757 | 1370 (let ((length (length bytes)) |
58210
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
1371 (ptr 0) optr tags op offset |
757 | 1372 lap tmp |
58210
2d6fd1fbee2e
(byte-optimize-inline-handler): Simplify.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
58165
diff
changeset
|
1373 endtag) |
757 | 1374 (while (not (= ptr length)) |
8294
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1375 (or make-spliceable |
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1376 (setq lap (cons ptr lap))) |
757 | 1377 (setq op (aref bytes ptr) |
1378 optr ptr | |
1379 offset (disassemble-offset)) ; this does dynamic-scope magic | |
1380 (setq op (aref byte-code-vector op)) | |
848 | 1381 (cond ((memq op byte-goto-ops) |
757 | 1382 ;; it's a pc |
1383 (setq offset | |
1384 (cdr (or (assq offset tags) | |
1385 (car (setq tags | |
1386 (cons (cons offset | |
1387 (byte-compile-make-tag)) | |
1388 tags))))))) | |
1389 ((cond ((eq op 'byte-constant2) (setq op 'byte-constant) t) | |
1390 ((memq op byte-constref-ops))) | |
22074
b52cdd6c996e
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
21590
diff
changeset
|
1391 (setq tmp (if (>= offset (length constvec)) |
b52cdd6c996e
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
21590
diff
changeset
|
1392 (list 'out-of-range offset) |
b52cdd6c996e
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
21590
diff
changeset
|
1393 (aref constvec offset)) |
757 | 1394 offset (if (eq op 'byte-constant) |
1395 (byte-compile-get-constant tmp) | |
1396 (or (assq tmp byte-compile-variables) | |
1397 (car (setq byte-compile-variables | |
1398 (cons (list tmp) | |
1399 byte-compile-variables))))))) | |
8294
cd3d2474ea10
(byte-decompile-bytecode-1): Don't add pc values
Richard M. Stallman <rms@gnu.org>
parents:
8292
diff
changeset
|
1400 ((and make-spliceable |
757 | 1401 (eq op 'byte-return)) |
1402 (if (= ptr (1- length)) | |
1403 (setq op nil) | |
1404 (setq offset (or endtag (setq endtag (byte-compile-make-tag))) | |
1405 op 'byte-goto)))) | |
1406 ;; lap = ( [ (pc . (op . arg)) ]* ) | |
1407 (setq lap (cons (cons optr (cons op (or offset 0))) | |
1408 lap)) | |
1409 (setq ptr (1+ ptr))) | |
1410 ;; take off the dummy nil op that we replaced a trailing "return" with. | |
1411 (let ((rest lap)) | |
1412 (while rest | |
8292
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1413 (cond ((numberp (car rest))) |
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1414 ((setq tmp (assq (car (car rest)) tags)) |
757 | 1415 ;; this addr is jumped to |
1416 (setcdr rest (cons (cons nil (cdr tmp)) | |
1417 (cdr rest))) | |
1418 (setq tags (delq tmp tags)) | |
1419 (setq rest (cdr rest)))) | |
1420 (setq rest (cdr rest)))) | |
1421 (if tags (error "optimizer error: missed tags %s" tags)) | |
1422 (if (null (car (cdr (car lap)))) | |
1423 (setq lap (cdr lap))) | |
1424 (if endtag | |
1425 (setq lap (cons (cons nil endtag) lap))) | |
1426 ;; remove addrs, lap = ( [ (op . arg) | (TAG tagno) ]* ) | |
8292
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1427 (mapcar (function (lambda (elt) |
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1428 (if (numberp elt) |
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1429 elt |
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1430 (cdr elt)))) |
6857db0f3c82
(byte-decompile-bytecode-1):
Richard M. Stallman <rms@gnu.org>
parents:
7298
diff
changeset
|
1431 (nreverse lap)))) |
757 | 1432 |
1433 | |
1434 ;;; peephole optimizer | |
1435 | |
1436 (defconst byte-tagref-ops (cons 'TAG byte-goto-ops)) | |
1437 | |
1438 (defconst byte-conditional-ops | |
1439 '(byte-goto-if-nil byte-goto-if-not-nil byte-goto-if-nil-else-pop | |
1440 byte-goto-if-not-nil-else-pop)) | |
1441 | |
1442 (defconst byte-after-unbind-ops | |
1443 '(byte-constant byte-dup | |
1444 byte-symbolp byte-consp byte-stringp byte-listp byte-numberp byte-integerp | |
21590
ef61a9126a73
(byte-after-unbind-ops): Delete byte-equal.
Richard M. Stallman <rms@gnu.org>
parents:
20874
diff
changeset
|
1445 byte-eq byte-not |
757 | 1446 byte-cons byte-list1 byte-list2 ; byte-list3 byte-list4 |
8466
3cca823100db
(byte-after-unbind-ops): Fix paren error wrt doc string.
Richard M. Stallman <rms@gnu.org>
parents:
8294
diff
changeset
|
1447 byte-interactive-p) |
3cca823100db
(byte-after-unbind-ops): Fix paren error wrt doc string.
Richard M. Stallman <rms@gnu.org>
parents:
8294
diff
changeset
|
1448 ;; How about other side-effect-free-ops? Is it safe to move an |
3cca823100db
(byte-after-unbind-ops): Fix paren error wrt doc string.
Richard M. Stallman <rms@gnu.org>
parents:
8294
diff
changeset
|
1449 ;; error invocation (such as from nth) out of an unwind-protect? |
21590
ef61a9126a73
(byte-after-unbind-ops): Delete byte-equal.
Richard M. Stallman <rms@gnu.org>
parents:
20874
diff
changeset
|
1450 ;; No, it is not, because the unwind-protect forms can alter |
ef61a9126a73
(byte-after-unbind-ops): Delete byte-equal.
Richard M. Stallman <rms@gnu.org>
parents:
20874
diff
changeset
|
1451 ;; the inside of the object to which nth would apply. |
ef61a9126a73
(byte-after-unbind-ops): Delete byte-equal.
Richard M. Stallman <rms@gnu.org>
parents:
20874
diff
changeset
|
1452 ;; For the same reason, byte-equal was deleted from this list. |
8466
3cca823100db
(byte-after-unbind-ops): Fix paren error wrt doc string.
Richard M. Stallman <rms@gnu.org>
parents:
8294
diff
changeset
|
1453 "Byte-codes that can be moved past an unbind.") |
757 | 1454 |
1455 (defconst byte-compile-side-effect-and-error-free-ops | |
1456 '(byte-constant byte-dup byte-symbolp byte-consp byte-stringp byte-listp | |
1457 byte-integerp byte-numberp byte-eq byte-equal byte-not byte-car-safe | |
1458 byte-cdr-safe byte-cons byte-list1 byte-list2 byte-point byte-point-max | |
1459 byte-point-min byte-following-char byte-preceding-char | |
1460 byte-current-column byte-eolp byte-eobp byte-bolp byte-bobp | |
1461 byte-current-buffer byte-interactive-p)) | |
1462 | |
1463 (defconst byte-compile-side-effect-free-ops | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
1464 (nconc |
757 | 1465 '(byte-varref byte-nth byte-memq byte-car byte-cdr byte-length byte-aref |
1466 byte-symbol-value byte-get byte-concat2 byte-concat3 byte-sub1 byte-add1 | |
1467 byte-eqlsign byte-gtr byte-lss byte-leq byte-geq byte-diff byte-negate | |
1468 byte-plus byte-max byte-min byte-mult byte-char-after byte-char-syntax | |
1469 byte-buffer-substring byte-string= byte-string< byte-nthcdr byte-elt | |
1470 byte-member byte-assq byte-quo byte-rem) | |
1471 byte-compile-side-effect-and-error-free-ops)) | |
1472 | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1473 ;; This crock is because of the way DEFVAR_BOOL variables work. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1474 ;; Consider the code |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1475 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1476 ;; (defun foo (flag) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1477 ;; (let ((old-pop-ups pop-up-windows) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1478 ;; (pop-up-windows flag)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1479 ;; (cond ((not (eq pop-up-windows old-pop-ups)) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1480 ;; (setq old-pop-ups pop-up-windows) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1481 ;; ...)))) |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1482 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1483 ;; Uncompiled, old-pop-ups will always be set to nil or t, even if FLAG is |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1484 ;; something else. But if we optimize |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1485 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1486 ;; varref flag |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1487 ;; varbind pop-up-windows |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1488 ;; varref pop-up-windows |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1489 ;; not |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1490 ;; to |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1491 ;; varref flag |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1492 ;; dup |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1493 ;; varbind pop-up-windows |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1494 ;; not |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1495 ;; |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1496 ;; we break the program, because it will appear that pop-up-windows and |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1497 ;; old-pop-ups are not EQ when really they are. So we have to know what |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1498 ;; the BOOL variables are, and not perform this optimization on them. |
25557
5eb4b90c57b0
(byte-boolean-vars): Removed. (Now primitive.)
Dave Love <fx@gnu.org>
parents:
25471
diff
changeset
|
1499 |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1500 ;; The variable `byte-boolean-vars' is now primitive and updated |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1501 ;; automatically by DEFVAR_BOOL. |
757 | 1502 |
1503 (defun byte-optimize-lapcode (lap &optional for-effect) | |
54495
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1504 "Simple peephole optimizer. LAP is both modified and returned. |
fec123d89bd0
(byte-compile-log-lap, byte-compile-inline-expand): Use backquote.
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
52401
diff
changeset
|
1505 If FOR-EFFECT is non-nil, the return value is assumed to be of no importance." |
32078
038a08ffb9f8
(byte-optimize-lapcode): Don't bind
Dave Love <fx@gnu.org>
parents:
29580
diff
changeset
|
1506 (let (lap0 |
038a08ffb9f8
(byte-optimize-lapcode): Don't bind
Dave Love <fx@gnu.org>
parents:
29580
diff
changeset
|
1507 lap1 |
038a08ffb9f8
(byte-optimize-lapcode): Don't bind
Dave Love <fx@gnu.org>
parents:
29580
diff
changeset
|
1508 lap2 |
757 | 1509 (keep-going 'first-time) |
1510 (add-depth 0) | |
1511 rest tmp tmp2 tmp3 | |
1512 (side-effect-free (if byte-compile-delete-errors | |
1513 byte-compile-side-effect-free-ops | |
1514 byte-compile-side-effect-and-error-free-ops))) | |
1515 (while keep-going | |
1516 (or (eq keep-going 'first-time) | |
1517 (byte-compile-log-lap " ---- next pass")) | |
1518 (setq rest lap | |
1519 keep-going nil) | |
1520 (while rest | |
1521 (setq lap0 (car rest) | |
1522 lap1 (nth 1 rest) | |
1523 lap2 (nth 2 rest)) | |
1524 | |
1525 ;; You may notice that sequences like "dup varset discard" are | |
1526 ;; optimized but sequences like "dup varset TAG1: discard" are not. | |
1527 ;; You may be tempted to change this; resist that temptation. | |
1528 (cond ;; | |
1529 ;; <side-effect-free> pop --> <deleted> | |
1530 ;; ...including: | |
1531 ;; const-X pop --> <deleted> | |
1532 ;; varref-X pop --> <deleted> | |
1533 ;; dup pop --> <deleted> | |
1534 ;; | |
1535 ((and (eq 'byte-discard (car lap1)) | |
1536 (memq (car lap0) side-effect-free)) | |
1537 (setq keep-going t) | |
1538 (setq tmp (aref byte-stack+-info (symbol-value (car lap0)))) | |
1539 (setq rest (cdr rest)) | |
1540 (cond ((= tmp 1) | |
1541 (byte-compile-log-lap | |
1542 " %s discard\t-->\t<deleted>" lap0) | |
1543 (setq lap (delq lap0 (delq lap1 lap)))) | |
1544 ((= tmp 0) | |
1545 (byte-compile-log-lap | |
1546 " %s discard\t-->\t<deleted> discard" lap0) | |
1547 (setq lap (delq lap0 lap))) | |
1548 ((= tmp -1) | |
1549 (byte-compile-log-lap | |
1550 " %s discard\t-->\tdiscard discard" lap0) | |
1551 (setcar lap0 'byte-discard) | |
1552 (setcdr lap0 0)) | |
1553 ((error "Optimizer error: too much on the stack")))) | |
1554 ;; | |
1555 ;; goto*-X X: --> X: | |
1556 ;; | |
1557 ((and (memq (car lap0) byte-goto-ops) | |
1558 (eq (cdr lap0) lap1)) | |
1559 (cond ((eq (car lap0) 'byte-goto) | |
1560 (setq lap (delq lap0 lap)) | |
1561 (setq tmp "<deleted>")) | |
1562 ((memq (car lap0) byte-goto-always-pop-ops) | |
1563 (setcar lap0 (setq tmp 'byte-discard)) | |
1564 (setcdr lap0 0)) | |
1565 ((error "Depth conflict at tag %d" (nth 2 lap0)))) | |
1566 (and (memq byte-optimize-log '(t byte)) | |
1567 (byte-compile-log " (goto %s) %s:\t-->\t%s %s:" | |
1568 (nth 1 lap1) (nth 1 lap1) | |
1569 tmp (nth 1 lap1))) | |
1570 (setq keep-going t)) | |
1571 ;; | |
1572 ;; varset-X varref-X --> dup varset-X | |
1573 ;; varbind-X varref-X --> dup varbind-X | |
1574 ;; const/dup varset-X varref-X --> const/dup varset-X const/dup | |
1575 ;; const/dup varbind-X varref-X --> const/dup varbind-X const/dup | |
1576 ;; The latter two can enable other optimizations. | |
1577 ;; | |
1578 ((and (eq 'byte-varref (car lap2)) | |
1579 (eq (cdr lap1) (cdr lap2)) | |
1580 (memq (car lap1) '(byte-varset byte-varbind))) | |
1581 (if (and (setq tmp (memq (car (cdr lap2)) byte-boolean-vars)) | |
1582 (not (eq (car lap0) 'byte-constant))) | |
1583 nil | |
1584 (setq keep-going t) | |
1585 (if (memq (car lap0) '(byte-constant byte-dup)) | |
1586 (progn | |
1587 (setq tmp (if (or (not tmp) | |
27823 | 1588 (byte-compile-const-symbol-p |
1589 (car (cdr lap0)))) | |
757 | 1590 (cdr lap0) |
1591 (byte-compile-get-constant t))) | |
1592 (byte-compile-log-lap " %s %s %s\t-->\t%s %s %s" | |
1593 lap0 lap1 lap2 lap0 lap1 | |
1594 (cons (car lap0) tmp)) | |
1595 (setcar lap2 (car lap0)) | |
1596 (setcdr lap2 tmp)) | |
1597 (byte-compile-log-lap " %s %s\t-->\tdup %s" lap1 lap2 lap1) | |
1598 (setcar lap2 (car lap1)) | |
1599 (setcar lap1 'byte-dup) | |
1600 (setcdr lap1 0) | |
1601 ;; The stack depth gets locally increased, so we will | |
1602 ;; increase maxdepth in case depth = maxdepth here. | |
1603 ;; This can cause the third argument to byte-code to | |
1604 ;; be larger than necessary. | |
1605 (setq add-depth 1)))) | |
1606 ;; | |
1607 ;; dup varset-X discard --> varset-X | |
1608 ;; dup varbind-X discard --> varbind-X | |
1609 ;; (the varbind variant can emerge from other optimizations) | |
1610 ;; | |
1611 ((and (eq 'byte-dup (car lap0)) | |
1612 (eq 'byte-discard (car lap2)) | |
1613 (memq (car lap1) '(byte-varset byte-varbind))) | |
1614 (byte-compile-log-lap " dup %s discard\t-->\t%s" lap1 lap1) | |
1615 (setq keep-going t | |
1616 rest (cdr rest)) | |
1617 (setq lap (delq lap0 (delq lap2 lap)))) | |
1618 ;; | |
1619 ;; not goto-X-if-nil --> goto-X-if-non-nil | |
1620 ;; not goto-X-if-non-nil --> goto-X-if-nil | |
1621 ;; | |
1622 ;; it is wrong to do the same thing for the -else-pop variants. | |
1623 ;; | |
1624 ((and (eq 'byte-not (car lap0)) | |
1625 (or (eq 'byte-goto-if-nil (car lap1)) | |
1626 (eq 'byte-goto-if-not-nil (car lap1)))) | |
1627 (byte-compile-log-lap " not %s\t-->\t%s" | |
1628 lap1 | |
1629 (cons | |
1630 (if (eq (car lap1) 'byte-goto-if-nil) | |
1631 'byte-goto-if-not-nil | |
1632 'byte-goto-if-nil) | |
1633 (cdr lap1))) | |
1634 (setcar lap1 (if (eq (car lap1) 'byte-goto-if-nil) | |
1635 'byte-goto-if-not-nil | |
1636 'byte-goto-if-nil)) | |
1637 (setq lap (delq lap0 lap)) | |
1638 (setq keep-going t)) | |
1639 ;; | |
1640 ;; goto-X-if-nil goto-Y X: --> goto-Y-if-non-nil X: | |
1641 ;; goto-X-if-non-nil goto-Y X: --> goto-Y-if-nil X: | |
1642 ;; | |
1643 ;; it is wrong to do the same thing for the -else-pop variants. | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
1644 ;; |
757 | 1645 ((and (or (eq 'byte-goto-if-nil (car lap0)) |
1646 (eq 'byte-goto-if-not-nil (car lap0))) ; gotoX | |
1647 (eq 'byte-goto (car lap1)) ; gotoY | |
1648 (eq (cdr lap0) lap2)) ; TAG X | |
1649 (let ((inverse (if (eq 'byte-goto-if-nil (car lap0)) | |
1650 'byte-goto-if-not-nil 'byte-goto-if-nil))) | |
1651 (byte-compile-log-lap " %s %s %s:\t-->\t%s %s:" | |
1652 lap0 lap1 lap2 | |
1653 (cons inverse (cdr lap1)) lap2) | |
1654 (setq lap (delq lap0 lap)) | |
1655 (setcar lap1 inverse) | |
1656 (setq keep-going t))) | |
1657 ;; | |
1658 ;; const goto-if-* --> whatever | |
1659 ;; | |
1660 ((and (eq 'byte-constant (car lap0)) | |
1661 (memq (car lap1) byte-conditional-ops)) | |
1662 (cond ((if (or (eq (car lap1) 'byte-goto-if-nil) | |
1663 (eq (car lap1) 'byte-goto-if-nil-else-pop)) | |
1664 (car (cdr lap0)) | |
1665 (not (car (cdr lap0)))) | |
1666 (byte-compile-log-lap " %s %s\t-->\t<deleted>" | |
1667 lap0 lap1) | |
1668 (setq rest (cdr rest) | |
1669 lap (delq lap0 (delq lap1 lap)))) | |
1670 (t | |
1671 (if (memq (car lap1) byte-goto-always-pop-ops) | |
1672 (progn | |
1673 (byte-compile-log-lap " %s %s\t-->\t%s" | |
1674 lap0 lap1 (cons 'byte-goto (cdr lap1))) | |
1675 (setq lap (delq lap0 lap))) | |
1676 (byte-compile-log-lap " %s %s\t-->\t%s" lap0 lap1 | |
1677 (cons 'byte-goto (cdr lap1)))) | |
1678 (setcar lap1 'byte-goto))) | |
1679 (setq keep-going t)) | |
1680 ;; | |
1681 ;; varref-X varref-X --> varref-X dup | |
1682 ;; varref-X [dup ...] varref-X --> varref-X [dup ...] dup | |
1683 ;; We don't optimize the const-X variations on this here, | |
1684 ;; because that would inhibit some goto optimizations; we | |
1685 ;; optimize the const-X case after all other optimizations. | |
1686 ;; | |
1687 ((and (eq 'byte-varref (car lap0)) | |
1688 (progn | |
1689 (setq tmp (cdr rest)) | |
1690 (while (eq (car (car tmp)) 'byte-dup) | |
1691 (setq tmp (cdr tmp))) | |
1692 t) | |
1693 (eq (cdr lap0) (cdr (car tmp))) | |
1694 (eq 'byte-varref (car (car tmp)))) | |
1695 (if (memq byte-optimize-log '(t byte)) | |
1696 (let ((str "")) | |
1697 (setq tmp2 (cdr rest)) | |
1698 (while (not (eq tmp tmp2)) | |
1699 (setq tmp2 (cdr tmp2) | |
1700 str (concat str " dup"))) | |
1701 (byte-compile-log-lap " %s%s %s\t-->\t%s%s dup" | |
1702 lap0 str lap0 lap0 str))) | |
1703 (setq keep-going t) | |
1704 (setcar (car tmp) 'byte-dup) | |
1705 (setcdr (car tmp) 0) | |
1706 (setq rest tmp)) | |
1707 ;; | |
1708 ;; TAG1: TAG2: --> TAG1: <deleted> | |
1709 ;; (and other references to TAG2 are replaced with TAG1) | |
1710 ;; | |
1711 ((and (eq (car lap0) 'TAG) | |
1712 (eq (car lap1) 'TAG)) | |
1713 (and (memq byte-optimize-log '(t byte)) | |
3591
507f64624555
Apply typo patches from Paul Eggert.
Jim Blandy <jimb@redhat.com>
parents:
3138
diff
changeset
|
1714 (byte-compile-log " adjacent tags %d and %d merged" |
757 | 1715 (nth 1 lap1) (nth 1 lap0))) |
1716 (setq tmp3 lap) | |
1717 (while (setq tmp2 (rassq lap0 tmp3)) | |
1718 (setcdr tmp2 lap1) | |
1719 (setq tmp3 (cdr (memq tmp2 tmp3)))) | |
1720 (setq lap (delq lap0 lap) | |
1721 keep-going t)) | |
1722 ;; | |
1723 ;; unused-TAG: --> <deleted> | |
1724 ;; | |
1725 ((and (eq 'TAG (car lap0)) | |
1726 (not (rassq lap0 lap))) | |
1727 (and (memq byte-optimize-log '(t byte)) | |
1728 (byte-compile-log " unused tag %d removed" (nth 1 lap0))) | |
1729 (setq lap (delq lap0 lap) | |
1730 keep-going t)) | |
1731 ;; | |
1732 ;; goto ... --> goto <delete until TAG or end> | |
1733 ;; return ... --> return <delete until TAG or end> | |
1734 ;; | |
1735 ((and (memq (car lap0) '(byte-goto byte-return)) | |
1736 (not (memq (car lap1) '(TAG nil)))) | |
1737 (setq tmp rest) | |
1738 (let ((i 0) | |
1739 (opt-p (memq byte-optimize-log '(t lap))) | |
1740 str deleted) | |
1741 (while (and (setq tmp (cdr tmp)) | |
1742 (not (eq 'TAG (car (car tmp))))) | |
1743 (if opt-p (setq deleted (cons (car tmp) deleted) | |
1744 str (concat str " %s") | |
1745 i (1+ i)))) | |
1746 (if opt-p | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
1747 (let ((tagstr |
757 | 1748 (if (eq 'TAG (car (car tmp))) |
12638
4adf53113ec9
(byte-optimize-lapcode): Fix format calls.
Richard M. Stallman <rms@gnu.org>
parents:
12550
diff
changeset
|
1749 (format "%d:" (car (cdr (car tmp)))) |
757 | 1750 (or (car tmp) "")))) |
1751 (if (< i 6) | |
1752 (apply 'byte-compile-log-lap-1 | |
1753 (concat " %s" str | |
1754 " %s\t-->\t%s <deleted> %s") | |
1755 lap0 | |
1756 (nconc (nreverse deleted) | |
1757 (list tagstr lap0 tagstr))) | |
1758 (byte-compile-log-lap | |
1759 " %s <%d unreachable op%s> %s\t-->\t%s <deleted> %s" | |
1760 lap0 i (if (= i 1) "" "s") | |
1761 tagstr lap0 tagstr)))) | |
1762 (rplacd rest tmp)) | |
1763 (setq keep-going t)) | |
1764 ;; | |
1765 ;; <safe-op> unbind --> unbind <safe-op> | |
1766 ;; (this may enable other optimizations.) | |
1767 ;; | |
1768 ((and (eq 'byte-unbind (car lap1)) | |
1769 (memq (car lap0) byte-after-unbind-ops)) | |
1770 (byte-compile-log-lap " %s %s\t-->\t%s %s" lap0 lap1 lap1 lap0) | |
1771 (setcar rest lap1) | |
1772 (setcar (cdr rest) lap0) | |
1773 (setq keep-going t)) | |
1774 ;; | |
1775 ;; varbind-X unbind-N --> discard unbind-(N-1) | |
1776 ;; save-excursion unbind-N --> unbind-(N-1) | |
1777 ;; save-restriction unbind-N --> unbind-(N-1) | |
1778 ;; | |
1779 ((and (eq 'byte-unbind (car lap1)) | |
1780 (memq (car lap0) '(byte-varbind byte-save-excursion | |
1781 byte-save-restriction)) | |
1782 (< 0 (cdr lap1))) | |
1783 (if (zerop (setcdr lap1 (1- (cdr lap1)))) | |
1784 (delq lap1 rest)) | |
1785 (if (eq (car lap0) 'byte-varbind) | |
1786 (setcar rest (cons 'byte-discard 0)) | |
1787 (setq lap (delq lap0 lap))) | |
1788 (byte-compile-log-lap " %s %s\t-->\t%s %s" | |
1789 lap0 (cons (car lap1) (1+ (cdr lap1))) | |
1790 (if (eq (car lap0) 'byte-varbind) | |
1791 (car rest) | |
1792 (car (cdr rest))) | |
1793 (if (and (/= 0 (cdr lap1)) | |
1794 (eq (car lap0) 'byte-varbind)) | |
1795 (car (cdr rest)) | |
1796 "")) | |
1797 (setq keep-going t)) | |
1798 ;; | |
1799 ;; goto*-X ... X: goto-Y --> goto*-Y | |
1800 ;; goto-X ... X: return --> return | |
1801 ;; | |
1802 ((and (memq (car lap0) byte-goto-ops) | |
1803 (memq (car (setq tmp (nth 1 (memq (cdr lap0) lap)))) | |
1804 '(byte-goto byte-return))) | |
1805 (cond ((and (not (eq tmp lap0)) | |
1806 (or (eq (car lap0) 'byte-goto) | |
1807 (eq (car tmp) 'byte-goto))) | |
1808 (byte-compile-log-lap " %s [%s]\t-->\t%s" | |
1809 (car lap0) tmp tmp) | |
1810 (if (eq (car tmp) 'byte-return) | |
1811 (setcar lap0 'byte-return)) | |
1812 (setcdr lap0 (cdr tmp)) | |
1813 (setq keep-going t)))) | |
1814 ;; | |
1815 ;; goto-*-else-pop X ... X: goto-if-* --> whatever | |
1816 ;; goto-*-else-pop X ... X: discard --> whatever | |
1817 ;; | |
1818 ((and (memq (car lap0) '(byte-goto-if-nil-else-pop | |
1819 byte-goto-if-not-nil-else-pop)) | |
1820 (memq (car (car (setq tmp (cdr (memq (cdr lap0) lap))))) | |
1821 (eval-when-compile | |
1822 (cons 'byte-discard byte-conditional-ops))) | |
1823 (not (eq lap0 (car tmp)))) | |
1824 (setq tmp2 (car tmp)) | |
1825 (setq tmp3 (assq (car lap0) '((byte-goto-if-nil-else-pop | |
1826 byte-goto-if-nil) | |
1827 (byte-goto-if-not-nil-else-pop | |
1828 byte-goto-if-not-nil)))) | |
1829 (if (memq (car tmp2) tmp3) | |
1830 (progn (setcar lap0 (car tmp2)) | |
1831 (setcdr lap0 (cdr tmp2)) | |
1832 (byte-compile-log-lap " %s-else-pop [%s]\t-->\t%s" | |
1833 (car lap0) tmp2 lap0)) | |
1834 ;; Get rid of the -else-pop's and jump one step further. | |
1835 (or (eq 'TAG (car (nth 1 tmp))) | |
1836 (setcdr tmp (cons (byte-compile-make-tag) | |
1837 (cdr tmp)))) | |
1838 (byte-compile-log-lap " %s [%s]\t-->\t%s <skip>" | |
1839 (car lap0) tmp2 (nth 1 tmp3)) | |
1840 (setcar lap0 (nth 1 tmp3)) | |
1841 (setcdr lap0 (nth 1 tmp))) | |
1842 (setq keep-going t)) | |
1843 ;; | |
1844 ;; const goto-X ... X: goto-if-* --> whatever | |
1845 ;; const goto-X ... X: discard --> whatever | |
1846 ;; | |
1847 ((and (eq (car lap0) 'byte-constant) | |
1848 (eq (car lap1) 'byte-goto) | |
1849 (memq (car (car (setq tmp (cdr (memq (cdr lap1) lap))))) | |
1850 (eval-when-compile | |
1851 (cons 'byte-discard byte-conditional-ops))) | |
1852 (not (eq lap1 (car tmp)))) | |
1853 (setq tmp2 (car tmp)) | |
1854 (cond ((memq (car tmp2) | |
1855 (if (null (car (cdr lap0))) | |
1856 '(byte-goto-if-nil byte-goto-if-nil-else-pop) | |
1857 '(byte-goto-if-not-nil | |
1858 byte-goto-if-not-nil-else-pop))) | |
1859 (byte-compile-log-lap " %s goto [%s]\t-->\t%s %s" | |
1860 lap0 tmp2 lap0 tmp2) | |
1861 (setcar lap1 (car tmp2)) | |
1862 (setcdr lap1 (cdr tmp2)) | |
1863 ;; Let next step fix the (const,goto-if*) sequence. | |
1864 (setq rest (cons nil rest))) | |
1865 (t | |
1866 ;; Jump one step further | |
1867 (byte-compile-log-lap | |
1868 " %s goto [%s]\t-->\t<deleted> goto <skip>" | |
1869 lap0 tmp2) | |
1870 (or (eq 'TAG (car (nth 1 tmp))) | |
1871 (setcdr tmp (cons (byte-compile-make-tag) | |
1872 (cdr tmp)))) | |
1873 (setcdr lap1 (car (cdr tmp))) | |
1874 (setq lap (delq lap0 lap)))) | |
1875 (setq keep-going t)) | |
1876 ;; | |
1877 ;; X: varref-Y ... varset-Y goto-X --> | |
1878 ;; X: varref-Y Z: ... dup varset-Y goto-Z | |
1879 ;; (varset-X goto-BACK, BACK: varref-X --> copy the varref down.) | |
1880 ;; (This is so usual for while loops that it is worth handling). | |
1881 ;; | |
1882 ((and (eq (car lap1) 'byte-varset) | |
1883 (eq (car lap2) 'byte-goto) | |
1884 (not (memq (cdr lap2) rest)) ;Backwards jump | |
1885 (eq (car (car (setq tmp (cdr (memq (cdr lap2) lap))))) | |
1886 'byte-varref) | |
1887 (eq (cdr (car tmp)) (cdr lap1)) | |
1888 (not (memq (car (cdr lap1)) byte-boolean-vars))) | |
1889 ;;(byte-compile-log-lap " Pulled %s to end of loop" (car tmp)) | |
1890 (let ((newtag (byte-compile-make-tag))) | |
1891 (byte-compile-log-lap | |
1892 " %s: %s ... %s %s\t-->\t%s: %s %s: ... %s %s %s" | |
1893 (nth 1 (cdr lap2)) (car tmp) | |
1894 lap1 lap2 | |
1895 (nth 1 (cdr lap2)) (car tmp) | |
1896 (nth 1 newtag) 'byte-dup lap1 | |
1897 (cons 'byte-goto newtag) | |
1898 ) | |
1899 (setcdr rest (cons (cons 'byte-dup 0) (cdr rest))) | |
1900 (setcdr tmp (cons (setcdr lap2 newtag) (cdr tmp)))) | |
1901 (setq add-depth 1) | |
1902 (setq keep-going t)) | |
1903 ;; | |
1904 ;; goto-X Y: ... X: goto-if*-Y --> goto-if-not-*-X+1 Y: | |
1905 ;; (This can pull the loop test to the end of the loop) | |
1906 ;; | |
1907 ((and (eq (car lap0) 'byte-goto) | |
1908 (eq (car lap1) 'TAG) | |
1909 (eq lap1 | |
1910 (cdr (car (setq tmp (cdr (memq (cdr lap0) lap)))))) | |
1911 (memq (car (car tmp)) | |
1912 '(byte-goto byte-goto-if-nil byte-goto-if-not-nil | |
1913 byte-goto-if-nil-else-pop))) | |
1914 ;; (byte-compile-log-lap " %s %s, %s %s --> moved conditional" | |
1915 ;; lap0 lap1 (cdr lap0) (car tmp)) | |
1916 (let ((newtag (byte-compile-make-tag))) | |
1917 (byte-compile-log-lap | |
1918 "%s %s: ... %s: %s\t-->\t%s ... %s:" | |
1919 lap0 (nth 1 lap1) (nth 1 (cdr lap0)) (car tmp) | |
1920 (cons (cdr (assq (car (car tmp)) | |
1921 '((byte-goto-if-nil . byte-goto-if-not-nil) | |
1922 (byte-goto-if-not-nil . byte-goto-if-nil) | |
1923 (byte-goto-if-nil-else-pop . | |
1924 byte-goto-if-not-nil-else-pop) | |
1925 (byte-goto-if-not-nil-else-pop . | |
1926 byte-goto-if-nil-else-pop)))) | |
1927 newtag) | |
49598
0d8b17d428b5
Trailing whitepace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
49041
diff
changeset
|
1928 |
757 | 1929 (nth 1 newtag) |
1930 ) | |
1931 (setcdr tmp (cons (setcdr lap0 newtag) (cdr tmp))) | |
1932 (if (eq (car (car tmp)) 'byte-goto-if-nil-else-pop) | |
1933 ;; We can handle this case but not the -if-not-nil case, | |
1934 ;; because we won't know which non-nil constant to push. | |
1935 (setcdr rest (cons (cons 'byte-constant | |
1936 (byte-compile-get-constant nil)) | |
1937 (cdr rest)))) | |
1938 (setcar lap0 (nth 1 (memq (car (car tmp)) | |
1939 '(byte-goto-if-nil-else-pop | |
1940 byte-goto-if-not-nil | |
1941 byte-goto-if-nil | |
1942 byte-goto-if-not-nil | |
1943 byte-goto byte-goto)))) | |
1944 ) | |
1945 (setq keep-going t)) | |
1946 ) | |
1947 (setq rest (cdr rest))) | |
1948 ) | |
1949 ;; Cleanup stage: | |
1950 ;; Rebuild byte-compile-constants / byte-compile-variables. | |
1951 ;; Simple optimizations that would inhibit other optimizations if they | |
1952 ;; were done in the optimizing loop, and optimizations which there is no | |
1953 ;; need to do more than once. | |
1954 (setq byte-compile-constants nil | |
1955 byte-compile-variables nil) | |
1956 (setq rest lap) | |
1957 (while rest | |
1958 (setq lap0 (car rest) | |
1959 lap1 (nth 1 rest)) | |
1960 (if (memq (car lap0) byte-constref-ops) | |
39777 | 1961 (if (or (eq (car lap0) 'byte-constant) |
1962 (eq (car lap0) 'byte-constant2)) | |
1963 (unless (memq (cdr lap0) byte-compile-constants) | |
757 | 1964 (setq byte-compile-constants (cons (cdr lap0) |
39777 | 1965 byte-compile-constants))) |
1966 (unless (memq (cdr lap0) byte-compile-variables) | |
1967 (setq byte-compile-variables (cons (cdr lap0) | |
1968 byte-compile-variables))))) | |
757 | 1969 (cond (;; |
1970 ;; const-C varset-X const-C --> const-C dup varset-X | |
1971 ;; const-C varbind-X const-C --> const-C dup varbind-X | |
1972 ;; | |
1973 (and (eq (car lap0) 'byte-constant) | |
1974 (eq (car (nth 2 rest)) 'byte-constant) | |
39777 | 1975 (eq (cdr lap0) (cdr (nth 2 rest))) |
757 | 1976 (memq (car lap1) '(byte-varbind byte-varset))) |
1977 (byte-compile-log-lap " %s %s %s\t-->\t%s dup %s" | |
1978 lap0 lap1 lap0 lap0 lap1) | |
1979 (setcar (cdr (cdr rest)) (cons (car lap1) (cdr lap1))) | |
1980 (setcar (cdr rest) (cons 'byte-dup 0)) | |
1981 (setq add-depth 1)) | |
1982 ;; | |
1983 ;; const-X [dup/const-X ...] --> const-X [dup ...] dup | |
1984 ;; varref-X [dup/varref-X ...] --> varref-X [dup ...] dup | |
1985 ;; | |
1986 ((memq (car lap0) '(byte-constant byte-varref)) | |
1987 (setq tmp rest | |
1988 tmp2 nil) | |
1989 (while (progn | |
1990 (while (eq 'byte-dup (car (car (setq tmp (cdr tmp)))))) | |
1991 (and (eq (cdr lap0) (cdr (car tmp))) | |
1992 (eq (car lap0) (car (car tmp))))) | |
1993 (setcar tmp (cons 'byte-dup 0)) | |
1994 (setq tmp2 t)) | |
1995 (if tmp2 | |
1996 (byte-compile-log-lap | |
12638
4adf53113ec9
(byte-optimize-lapcode): Fix format calls.
Richard M. Stallman <rms@gnu.org>
parents:
12550
diff
changeset
|
1997 " %s [dup/%s]...\t-->\t%s dup..." lap0 lap0 lap0))) |
757 | 1998 ;; |
1999 ;; unbind-N unbind-M --> unbind-(N+M) | |
2000 ;; | |
2001 ((and (eq 'byte-unbind (car lap0)) | |
2002 (eq 'byte-unbind (car lap1))) | |
2003 (byte-compile-log-lap " %s %s\t-->\t%s" lap0 lap1 | |
2004 (cons 'byte-unbind | |
2005 (+ (cdr lap0) (cdr lap1)))) | |
2006 (setq keep-going t) | |
2007 (setq lap (delq lap0 lap)) | |
2008 (setcdr lap1 (+ (cdr lap1) (cdr lap0)))) | |
2009 ) | |
2010 (setq rest (cdr rest))) | |
2011 (setq byte-compile-maxdepth (+ byte-compile-maxdepth add-depth))) | |
2012 lap) | |
2013 | |
25231
b587c4aeff78
Provide `byte-optimize', not `byte-opt'.
Karl Heuer <kwzh@gnu.org>
parents:
24734
diff
changeset
|
2014 (provide 'byte-opt) |
757 | 2015 |
2016 | |
2017 ;; To avoid "lisp nesting exceeds max-lisp-eval-depth" when this file compiles | |
2018 ;; itself, compile some of its most used recursive functions (at load time). | |
2019 ;; | |
2020 (eval-when-compile | |
1818
7e3322619e46
compiled-function-p has been renamed to byte-code-function-p.
Jim Blandy <jimb@redhat.com>
parents:
957
diff
changeset
|
2021 (or (byte-code-function-p (symbol-function 'byte-optimize-form)) |
757 | 2022 (assq 'byte-code (symbol-function 'byte-optimize-form)) |
2023 (let ((byte-optimize nil) | |
2024 (byte-compile-warnings nil)) | |
85183
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2025 (mapc (lambda (x) |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2026 (or noninteractive (message "compiling %s..." x)) |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2027 (byte-compile x) |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2028 (or noninteractive (message "compiling %s...done" x))) |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2029 '(byte-optimize-form |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2030 byte-optimize-body |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2031 byte-optimize-predicate |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2032 byte-optimize-binary-predicate |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2033 ;; Inserted some more than necessary, to speed it up. |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2034 byte-optimize-form-code-walker |
d18b96e2d51f
(top level): Use mapc rather than mapcar.
Glenn Morris <rgm@gnu.org>
parents:
82907
diff
changeset
|
2035 byte-optimize-lapcode)))) |
757 | 2036 nil) |
848 | 2037 |
58165
cc9f1e89279b
(byte-compile-inline-expand): Understand the
Stefan Monnier <monnier@iro.umontreal.ca>
parents:
57423
diff
changeset
|
2038 ;; arch-tag: 0f14076b-737e-4bef-aae6-908826ec1ff1 |
848 | 2039 ;;; byte-opt.el ends here |