1.0.11.34: better SUBSEQ on lists
[sbcl/simd.git] / src / compiler / seqtran.lisp
blobd7dbab224908a162dcaa043e9a9380439a1fd197
1 ;;;; optimizers for list and sequence functions
3 ;;;; This software is part of the SBCL system. See the README file for
4 ;;;; more information.
5 ;;;;
6 ;;;; This software is derived from the CMU CL system, which was
7 ;;;; written at Carnegie Mellon University and released into the
8 ;;;; public domain. The software is in the public domain and is
9 ;;;; provided with absolutely no warranty. See the COPYING and CREDITS
10 ;;;; files for more information.
12 (in-package "SB!C")
14 ;;;; mapping onto lists: the MAPFOO functions
16 (defun mapfoo-transform (fn arglists accumulate take-car)
17 (collect ((do-clauses)
18 (args-to-fn)
19 (tests))
20 (let ((n-first (gensym)))
21 (dolist (a (if accumulate
22 arglists
23 `(,n-first ,@(rest arglists))))
24 (let ((v (gensym)))
25 (do-clauses `(,v ,a (cdr ,v)))
26 (tests `(endp ,v))
27 (args-to-fn (if take-car `(car ,v) v))))
29 (let* ((fn-sym (gensym)) ; for ONCE-ONLY-ish purposes
30 (call `(%funcall ,fn-sym . ,(args-to-fn)))
31 (endtest `(or ,@(tests))))
33 `(let ((,fn-sym (%coerce-callable-to-fun ,fn)))
34 ,(ecase accumulate
35 (:nconc
36 (let ((temp (gensym))
37 (map-result (gensym)))
38 `(let ((,map-result (list nil)))
39 (do-anonymous ((,temp ,map-result) . ,(do-clauses))
40 (,endtest (cdr ,map-result))
41 (setq ,temp (last (nconc ,temp ,call)))))))
42 (:list
43 (let ((temp (gensym))
44 (map-result (gensym)))
45 `(let ((,map-result (list nil)))
46 (do-anonymous ((,temp ,map-result) . ,(do-clauses))
47 (,endtest (truly-the list (cdr ,map-result)))
48 (rplacd ,temp (setq ,temp (list ,call)))))))
49 ((nil)
50 `(let ((,n-first ,(first arglists)))
51 (do-anonymous ,(do-clauses)
52 (,endtest (truly-the list ,n-first))
53 ,call)))))))))
55 (define-source-transform mapc (function list &rest more-lists)
56 (mapfoo-transform function (cons list more-lists) nil t))
58 (define-source-transform mapcar (function list &rest more-lists)
59 (mapfoo-transform function (cons list more-lists) :list t))
61 (define-source-transform mapcan (function list &rest more-lists)
62 (mapfoo-transform function (cons list more-lists) :nconc t))
64 (define-source-transform mapl (function list &rest more-lists)
65 (mapfoo-transform function (cons list more-lists) nil nil))
67 (define-source-transform maplist (function list &rest more-lists)
68 (mapfoo-transform function (cons list more-lists) :list nil))
70 (define-source-transform mapcon (function list &rest more-lists)
71 (mapfoo-transform function (cons list more-lists) :nconc nil))
73 ;;;; mapping onto sequences: the MAP function
75 ;;; MAP is %MAP plus a check to make sure that any length specified in
76 ;;; the result type matches the actual result. We also wrap it in a
77 ;;; TRULY-THE for the most specific type we can determine.
78 (deftransform map ((result-type-arg fun seq &rest seqs) * * :node node)
79 (let* ((seq-names (make-gensym-list (1+ (length seqs))))
80 (bare `(%map result-type-arg fun ,@seq-names))
81 (constant-result-type-arg-p (constant-lvar-p result-type-arg))
82 ;; what we know about the type of the result. (Note that the
83 ;; "result type" argument is not necessarily the type of the
84 ;; result, since NIL means the result has NULL type.)
85 (result-type (if (not constant-result-type-arg-p)
86 'consed-sequence
87 (let ((result-type-arg-value
88 (lvar-value result-type-arg)))
89 (if (null result-type-arg-value)
90 'null
91 result-type-arg-value)))))
92 `(lambda (result-type-arg fun ,@seq-names)
93 (truly-the ,result-type
94 ,(cond ((policy node (< safety 3))
95 ;; ANSI requires the length-related type check only
96 ;; when the SAFETY quality is 3... in other cases, we
97 ;; skip it, because it could be expensive.
98 bare)
99 ((not constant-result-type-arg-p)
100 `(sequence-of-checked-length-given-type ,bare
101 result-type-arg))
103 (let ((result-ctype (ir1-transform-specifier-type
104 result-type)))
105 (if (array-type-p result-ctype)
106 (let ((dims (array-type-dimensions result-ctype)))
107 (unless (and (listp dims) (= (length dims) 1))
108 (give-up-ir1-transform "invalid sequence type"))
109 (let ((dim (first dims)))
110 (if (eq dim '*)
111 bare
112 `(vector-of-checked-length-given-length ,bare
113 ,dim))))
114 ;; FIXME: this is wrong, as not all subtypes of
115 ;; VECTOR are ARRAY-TYPEs [consider, for
116 ;; example, (OR (VECTOR T 3) (VECTOR T
117 ;; 4))]. However, it's difficult to see what we
118 ;; should put here... maybe we should
119 ;; GIVE-UP-IR1-TRANSFORM if the type is a
120 ;; subtype of VECTOR but not an ARRAY-TYPE?
121 bare))))))))
123 ;;; Return a DO loop, mapping a function FUN to elements of
124 ;;; sequences. SEQS is a list of lvars, SEQ-NAMES - list of variables,
125 ;;; bound to sequences, INTO - a variable, which is used in
126 ;;; MAP-INTO. RESULT and BODY are forms, which can use variables
127 ;;; FUNCALL-RESULT, containing the result of application of FUN, and
128 ;;; INDEX, containing the current position in sequences.
129 (defun build-sequence-iterator (seqs seq-names &key result into body)
130 (declare (type list seqs seq-names)
131 (type symbol into))
132 (collect ((bindings)
133 (declarations)
134 (vector-lengths)
135 (tests)
136 (places))
137 (let ((found-vector-p nil))
138 (flet ((process-vector (length)
139 (unless found-vector-p
140 (setq found-vector-p t)
141 (bindings `(index 0 (1+ index)))
142 (declarations `(type index index)))
143 (vector-lengths length)))
144 (loop for seq of-type lvar in seqs
145 for seq-name in seq-names
146 for type = (lvar-type seq)
147 do (cond ((csubtypep type (specifier-type 'list))
148 (with-unique-names (index)
149 (bindings `(,index ,seq-name (cdr ,index)))
150 (declarations `(type list ,index))
151 (places `(car ,index))
152 (tests `(endp ,index))))
153 ((csubtypep type (specifier-type 'vector))
154 (process-vector `(length ,seq-name))
155 (places `(locally (declare (optimize (insert-array-bounds-checks 0)))
156 (aref ,seq-name index))))
158 (give-up-ir1-transform
159 "can't determine sequence argument type"))))
160 (when into
161 (process-vector `(array-dimension ,into 0))))
162 (when found-vector-p
163 (bindings `(length (min ,@(vector-lengths))))
164 (tests `(>= index length)))
165 `(do (,@(bindings))
166 ((or ,@(tests)) ,result)
167 (declare ,@(declarations))
168 (let ((funcall-result (funcall fun ,@(places))))
169 (declare (ignorable funcall-result))
170 ,body)))))
172 ;;; Try to compile %MAP efficiently when we can determine sequence
173 ;;; argument types at compile time.
175 ;;; Note: This transform was written to allow open coding of
176 ;;; quantifiers by expressing them in terms of (MAP NIL ..). For
177 ;;; non-NIL values of RESULT-TYPE, it's still useful, but not
178 ;;; necessarily as efficient as possible. In particular, it will be
179 ;;; inefficient when RESULT-TYPE is a SIMPLE-ARRAY with specialized
180 ;;; numeric element types. It should be straightforward to make it
181 ;;; handle that case more efficiently, but it's left as an exercise to
182 ;;; the reader, because the code is complicated enough already and I
183 ;;; don't happen to need that functionality right now. -- WHN 20000410
184 (deftransform %map ((result-type fun seq &rest seqs) * *
185 :policy (>= speed space))
186 "open code"
187 (unless (constant-lvar-p result-type)
188 (give-up-ir1-transform "RESULT-TYPE argument not constant"))
189 (labels ( ;; 1-valued SUBTYPEP, fails unless second value of SUBTYPEP is true
190 (fn-1subtypep (fn x y)
191 (multiple-value-bind (subtype-p valid-p) (funcall fn x y)
192 (if valid-p
193 subtype-p
194 (give-up-ir1-transform
195 "can't analyze sequence type relationship"))))
196 (1subtypep (x y) (fn-1subtypep #'sb!xc:subtypep x y)))
197 (let* ((result-type-value (lvar-value result-type))
198 (result-supertype (cond ((null result-type-value) 'null)
199 ((1subtypep result-type-value 'vector)
200 'vector)
201 ((1subtypep result-type-value 'list)
202 'list)
204 (give-up-ir1-transform
205 "result type unsuitable")))))
206 (cond ((and result-type-value (null seqs))
207 ;; The consing arity-1 cases can be implemented
208 ;; reasonably efficiently as function calls, and the cost
209 ;; of consing should be significantly larger than
210 ;; function call overhead, so we always compile these
211 ;; cases as full calls regardless of speed-versus-space
212 ;; optimization policy.
213 (cond ((subtypep result-type-value 'list)
214 '(%map-to-list-arity-1 fun seq))
215 ( ;; (This one can be inefficient due to COERCE, but
216 ;; the current open-coded implementation has the
217 ;; same problem.)
218 (subtypep result-type-value 'vector)
219 `(coerce (%map-to-simple-vector-arity-1 fun seq)
220 ',result-type-value))
221 (t (bug "impossible (?) sequence type"))))
223 (let* ((seqs (cons seq seqs))
224 (seq-args (make-gensym-list (length seqs))))
225 (multiple-value-bind (push-dacc result)
226 (ecase result-supertype
227 (null (values nil nil))
228 (list (values `(push funcall-result acc)
229 `(nreverse acc)))
230 (vector (values `(push funcall-result acc)
231 `(coerce (nreverse acc)
232 ',result-type-value))))
233 ;; (We use the same idiom, of returning a LAMBDA from
234 ;; DEFTRANSFORM, as is used in the DEFTRANSFORMs for
235 ;; FUNCALL and ALIEN-FUNCALL, and for the same
236 ;; reason: we need to get the runtime values of each
237 ;; of the &REST vars.)
238 `(lambda (result-type fun ,@seq-args)
239 (declare (ignore result-type))
240 (let ((fun (%coerce-callable-to-fun fun))
241 (acc nil))
242 (declare (type list acc))
243 (declare (ignorable acc))
244 ,(build-sequence-iterator
245 seqs seq-args
246 :result result
247 :body push-dacc))))))))))
249 ;;; MAP-INTO
250 (deftransform map-into ((result fun &rest seqs)
251 (vector * &rest *)
253 "open code"
254 (let ((seqs-names (mapcar (lambda (x)
255 (declare (ignore x))
256 (gensym))
257 seqs)))
258 `(lambda (result fun ,@seqs-names)
259 ,(build-sequence-iterator
260 seqs seqs-names
261 :result '(when (array-has-fill-pointer-p result)
262 (setf (fill-pointer result) index))
263 :into 'result
264 :body '(locally (declare (optimize (insert-array-bounds-checks 0)))
265 (setf (aref result index) funcall-result)))
266 result)))
269 ;;; FIXME: once the confusion over doing transforms with known-complex
270 ;;; arrays is over, we should also transform the calls to (AND (ARRAY
271 ;;; * (*)) (NOT (SIMPLE-ARRAY * (*)))) objects.
272 (deftransform elt ((s i) ((simple-array * (*)) *) *)
273 '(aref s i))
275 (deftransform elt ((s i) (list *) * :policy (< safety 3))
276 '(nth i s))
278 (deftransform %setelt ((s i v) ((simple-array * (*)) * *) *)
279 '(%aset s i v))
281 (deftransform %setelt ((s i v) (list * *) * :policy (< safety 3))
282 '(setf (car (nthcdr i s)) v))
284 (deftransform %check-vector-sequence-bounds ((vector start end)
285 (vector * *) *
286 :node node)
287 ;; FIXME: Should this not be INSERT-ARRAY-BOUNDS-CHECKS?
288 (if (policy node (< safety speed))
289 '(or end (length vector))
290 '(let ((length (length vector)))
291 (if (<= 0 start (or end length) length)
292 (or end length)
293 (sb!impl::signal-bounding-indices-bad-error vector start end)))))
295 (defun specialized-list-seek-function-name (function-name key-functions)
296 (or (find-symbol (with-output-to-string (s)
297 ;; Write "%NAME-FUN1-FUN2-FUN3", etc. Not only is
298 ;; this ever so slightly faster then FORMAT, this
299 ;; way we are also proof against *PRINT-CASE*
300 ;; frobbing and such.
301 (write-char #\% s)
302 (write-string (symbol-name function-name) s)
303 (dolist (f key-functions)
304 (write-char #\- s)
305 (write-string (symbol-name f) s)))
306 (load-time-value (find-package "SB!KERNEL")))
307 (bug "Unknown list item seek transform: name=~S, key-functions=~S"
308 function-name key-functions)))
310 (defun transform-list-item-seek (name list key test test-not node)
311 ;; Key can legally be NIL, but if it's NIL for sure we pretend it's
312 ;; not there at all. If it might be NIL, make up a form to that
313 ;; ensure it is a function.
314 (multiple-value-bind (key key-form)
315 (if key
316 (let ((key-type (lvar-type key))
317 (null-type (specifier-type 'null)))
318 (cond ((csubtypep key-type null-type)
319 (values nil nil))
320 ((csubtypep null-type key-type)
321 (values key '(if key
322 (%coerce-callable-to-fun key)
323 #'identity)))
325 (values key '(%coerce-callable-to-fun key))))))
326 (let* ((funs (remove nil (list (and key 'key) (cond (test 'test)
327 (test-not 'test-not)))))
328 (target-expr (if key '(%funcall key target) 'target))
329 (test-expr (cond (test `(%funcall test item ,target-expr))
330 (test-not `(not (%funcall test-not item ,target-expr)))
331 (t `(eql item ,target-expr)))))
332 (labels ((open-code (tail)
333 (when tail
334 `(if (let ((this ',(car tail)))
335 ,(ecase name
336 (assoc
337 `(and this (let ((target (car this)))
338 ,test-expr)))
339 (member
340 `(let ((target this))
341 ,test-expr))))
342 ',(ecase name
343 (assoc (car tail))
344 (member tail))
345 ,(open-code (cdr tail)))))
346 (ensure-fun (fun)
347 (if (eq 'key fun)
348 key-form
349 `(%coerce-callable-to-fun ,fun))))
350 (let* ((cp (constant-lvar-p list))
351 (c-list (when cp (lvar-value list))))
352 (cond ((and cp c-list (policy node (>= speed space)))
353 `(let ,(mapcar (lambda (fun) `(,fun ,(ensure-fun fun))) funs)
354 ,(open-code c-list)))
355 ((and cp (not c-list))
356 ;; constant nil list -- nothing to find!
357 nil)
359 ;; specialized out-of-line version
360 `(,(specialized-list-seek-function-name name funs)
361 item list ,@(mapcar #'ensure-fun funs)))))))))
363 (deftransform member ((item list &key key test test-not) * * :node node)
364 (transform-list-item-seek 'member list key test test-not node))
366 (deftransform assoc ((item list &key key test test-not) * * :node node)
367 (transform-list-item-seek 'assoc list key test test-not node))
369 (deftransform memq ((item list) (t (constant-arg list)))
370 (labels ((rec (tail)
371 (if tail
372 `(if (eq item ',(car tail))
373 ',tail
374 ,(rec (cdr tail)))
375 nil)))
376 (rec (lvar-value list))))
378 ;;; A similar transform used to apply to MEMBER and ASSOC, but since
379 ;;; TRANSFORM-LIST-ITEM-SEEK now takes care of them those transform
380 ;;; would never fire, and (%MEMBER-TEST ITEM LIST #'EQ) should be
381 ;;; almost as fast as MEMQ.
382 (deftransform delete ((item list &key test) (t list &rest t) *)
383 "convert to EQ test"
384 ;; FIXME: The scope of this transformation could be
385 ;; widened somewhat, letting it work whenever the test is
386 ;; 'EQL and we know from the type of ITEM that it #'EQ
387 ;; works like #'EQL on it. (E.g. types FIXNUM, CHARACTER,
388 ;; and SYMBOL.)
389 ;; If TEST is EQ, apply transform, else
390 ;; if test is not EQL, then give up on transform, else
391 ;; if ITEM is not a NUMBER or is a FIXNUM, apply
392 ;; transform, else give up on transform.
393 (cond (test
394 (unless (lvar-fun-is test '(eq))
395 (give-up-ir1-transform)))
396 ((types-equal-or-intersect (lvar-type item)
397 (specifier-type 'number))
398 (give-up-ir1-transform "Item might be a number.")))
399 `(delq item list))
401 (deftransform delete-if ((pred list) (t list))
402 "open code"
403 '(do ((x list (cdr x))
404 (splice '()))
405 ((endp x) list)
406 (cond ((funcall pred (car x))
407 (if (null splice)
408 (setq list (cdr x))
409 (rplacd splice (cdr x))))
410 (t (setq splice x)))))
412 (deftransform fill ((seq item &key (start 0) (end (length seq)))
413 (vector t &key (:start t) (:end index))
415 :policy (> speed space))
416 "open code"
417 (let ((element-type (upgraded-element-type-specifier-or-give-up seq)))
418 (values
419 `(with-array-data ((data seq)
420 (start start)
421 (end end))
422 (declare (type (simple-array ,element-type 1) data))
423 (declare (type fixnum start end))
424 (do ((i start (1+ i)))
425 ((= i end) seq)
426 (declare (type index i))
427 ;; WITH-ARRAY-DATA did our range checks once and for all, so
428 ;; it'd be wasteful to check again on every AREF...
429 (declare (optimize (safety 0)))
430 (setf (aref data i) item)))
431 ;; ... though we still need to check that the new element can fit
432 ;; into the vector in safe code. -- CSR, 2002-07-05
433 `((declare (type ,element-type item))))))
435 ;;;; utilities
437 ;;; Return true if LVAR's only use is a non-NOTINLINE reference to a
438 ;;; global function with one of the specified NAMES.
439 (defun lvar-fun-is (lvar names)
440 (declare (type lvar lvar) (list names))
441 (let ((use (lvar-uses lvar)))
442 (and (ref-p use)
443 (let ((leaf (ref-leaf use)))
444 (and (global-var-p leaf)
445 (eq (global-var-kind leaf) :global-function)
446 (not (null (member (leaf-source-name leaf) names
447 :test #'equal))))))))
449 ;;; If LVAR is a constant lvar, the return the constant value. If it
450 ;;; is null, then return default, otherwise quietly give up the IR1
451 ;;; transform.
453 ;;; ### Probably should take an ARG and flame using the NAME.
454 (defun constant-value-or-lose (lvar &optional default)
455 (declare (type (or lvar null) lvar))
456 (cond ((not lvar) default)
457 ((constant-lvar-p lvar)
458 (lvar-value lvar))
460 (give-up-ir1-transform))))
463 ;;;; hairy sequence transforms
465 ;;; FIXME: no hairy sequence transforms in SBCL?
467 ;;; There used to be a bunch of commented out code about here,
468 ;;; containing the (apparent) beginning of hairy sequence transform
469 ;;; infrastructure. People interested in implementing better sequence
470 ;;; transforms might want to look at it for inspiration, even though
471 ;;; the actual code is ancient CMUCL -- and hence bitrotted. The code
472 ;;; was deleted in 1.0.7.23.
474 ;;;; string operations
476 ;;; We transform the case-sensitive string predicates into a non-keyword
477 ;;; version. This is an IR1 transform so that we don't have to worry about
478 ;;; changing the order of evaluation.
479 (macrolet ((def (fun pred*)
480 `(deftransform ,fun ((string1 string2 &key (start1 0) end1
481 (start2 0) end2)
482 * *)
483 `(,',pred* string1 string2 start1 end1 start2 end2))))
484 (def string< string<*)
485 (def string> string>*)
486 (def string<= string<=*)
487 (def string>= string>=*)
488 (def string= string=*)
489 (def string/= string/=*))
491 ;;; Return a form that tests the free variables STRING1 and STRING2
492 ;;; for the ordering relationship specified by LESSP and EQUALP. The
493 ;;; start and end are also gotten from the environment. Both strings
494 ;;; must be SIMPLE-BASE-STRINGs.
495 (macrolet ((def (name lessp equalp)
496 `(deftransform ,name ((string1 string2 start1 end1 start2 end2)
497 (simple-base-string simple-base-string t t t t) *)
498 `(let* ((end1 (if (not end1) (length string1) end1))
499 (end2 (if (not end2) (length string2) end2))
500 (index (sb!impl::%sp-string-compare
501 string1 start1 end1 string2 start2 end2)))
502 (if index
503 (cond ((= index end1)
504 ,(if ',lessp 'index nil))
505 ((= (+ index (- start2 start1)) end2)
506 ,(if ',lessp nil 'index))
507 ((,(if ',lessp 'char< 'char>)
508 (schar string1 index)
509 (schar string2
510 (truly-the index
511 (+ index
512 (truly-the fixnum
513 (- start2
514 start1))))))
515 index)
516 (t nil))
517 ,(if ',equalp 'end1 nil))))))
518 (def string<* t nil)
519 (def string<=* t t)
520 (def string>* nil nil)
521 (def string>=* nil t))
523 (macrolet ((def (name result-fun)
524 `(deftransform ,name ((string1 string2 start1 end1 start2 end2)
525 (simple-base-string simple-base-string t t t t) *)
526 `(,',result-fun
527 (sb!impl::%sp-string-compare
528 string1 start1 (or end1 (length string1))
529 string2 start2 (or end2 (length string2)))))))
530 (def string=* not)
531 (def string/=* identity))
534 ;;;; transforms for sequence functions
536 ;;; Moved here from generic/vm-tran.lisp to satisfy clisp. Only applies
537 ;;; to vectors based on simple arrays.
538 (def!constant vector-data-bit-offset
539 (* sb!vm:vector-data-offset sb!vm:n-word-bits))
541 (eval-when (:compile-toplevel)
542 (defun valid-bit-bash-saetp-p (saetp)
543 ;; BIT-BASHing isn't allowed on simple vectors that contain pointers
544 (and (not (eq t (sb!vm:saetp-specifier saetp)))
545 ;; Disallowing (VECTOR NIL) also means that we won't transform
546 ;; sequence functions into bit-bashing code and we let the
547 ;; generic sequence functions signal errors if necessary.
548 (not (zerop (sb!vm:saetp-n-bits saetp)))
549 ;; Due to limitations with the current BIT-BASHing code, we can't
550 ;; BIT-BASH reliably on arrays whose element types are larger
551 ;; than the word size.
552 (<= (sb!vm:saetp-n-bits saetp) sb!vm:n-word-bits)))
553 ) ; EVAL-WHEN
555 ;;; FIXME: In the copy loops below, we code the loops in a strange
556 ;;; fashion:
558 ;;; (do ((i (+ src-offset length) (1- i)))
559 ;;; ((<= i 0) ...)
560 ;;; (... (aref foo (1- i)) ...))
562 ;;; rather than the more natural (and seemingly more efficient):
564 ;;; (do ((i (1- (+ src-offset length)) (1- i)))
565 ;;; ((< i 0) ...)
566 ;;; (... (aref foo i) ...))
568 ;;; (more efficient because we don't have to do the index adjusting on
569 ;;; every iteration of the loop)
571 ;;; We do this to avoid a suboptimality in SBCL's backend. In the
572 ;;; latter case, the backend thinks I is a FIXNUM (which it is), but
573 ;;; when used as an array index, the backend thinks I is a
574 ;;; POSITIVE-FIXNUM (which it is). However, since the backend thinks of
575 ;;; these as distinct storage classes, it cannot coerce a move from a
576 ;;; FIXNUM TN to a POSITIVE-FIXNUM TN. The practical effect of this
577 ;;; deficiency is that we have two extra moves and increased register
578 ;;; pressure, which can lead to some spectacularly bad register
579 ;;; allocation. (sub-FIXME: the register allocation even with the
580 ;;; strangely written loops is not always excellent, either...). Doing
581 ;;; it the first way, above, means that I is always thought of as a
582 ;;; POSITIVE-FIXNUM and there are no issues.
584 ;;; Besides, the *-WITH-OFFSET machinery will fold those index
585 ;;; adjustments in the first version into the array addressing at no
586 ;;; performance penalty!
588 ;;; This transform is critical to the performance of string streams. If
589 ;;; you tweak it, make sure that you compare the disassembly, if not the
590 ;;; performance of, the functions implementing string streams
591 ;;; (e.g. SB!IMPL::STRING-OUCH).
592 (eval-when (:compile-toplevel :load-toplevel :execute)
593 (defun make-replace-transform (saetp sequence-type1 sequence-type2)
594 `(deftransform replace ((seq1 seq2 &key (start1 0) (start2 0) end1 end2)
595 (,sequence-type1 ,sequence-type2 &rest t)
596 ,sequence-type1
597 :node node)
598 ,(cond
599 ((and saetp (valid-bit-bash-saetp-p saetp)) nil)
600 ;; If the sequence types are different, SEQ1 and SEQ2 must
601 ;; be distinct arrays, and we can open code the copy loop.
602 ((not (eql sequence-type1 sequence-type2)) nil)
603 ;; If we're not bit-bashing, only allow cases where we
604 ;; can determine the order of copying up front. (There
605 ;; are actually more cases we can handle if we know the
606 ;; amount that we're copying, but this handles the
607 ;; common cases.)
608 (t '(unless (= (constant-value-or-lose start1 0)
609 (constant-value-or-lose start2 0))
610 (give-up-ir1-transform))))
611 `(let* ((len1 (length seq1))
612 (len2 (length seq2))
613 (end1 (or end1 len1))
614 (end2 (or end2 len2))
615 (replace-len1 (- end1 start1))
616 (replace-len2 (- end2 start2)))
617 ,(unless (policy node (= safety 0))
618 `(progn
619 (unless (<= 0 start1 end1 len1)
620 (sb!impl::signal-bounding-indices-bad-error seq1 start1 end1))
621 (unless (<= 0 start2 end2 len2)
622 (sb!impl::signal-bounding-indices-bad-error seq2 start2 end2))))
623 ,',(cond
624 ((and saetp (valid-bit-bash-saetp-p saetp))
625 (let* ((n-element-bits (sb!vm:saetp-n-bits saetp))
626 (bash-function (intern (format nil "UB~D-BASH-COPY"
627 n-element-bits)
628 (find-package "SB!KERNEL"))))
629 `(funcall (function ,bash-function) seq2 start2
630 seq1 start1 (min replace-len1 replace-len2))))
632 ;; We can expand the loop inline here because we
633 ;; would have given up the transform (see above)
634 ;; if we didn't have constant matching start
635 ;; indices.
636 '(do ((i start1 (1+ i))
637 (j start2 (1+ j))
638 (end (+ start1
639 (min replace-len1 replace-len2))))
640 ((>= i end))
641 (declare (optimize (insert-array-bounds-checks 0)))
642 (setf (aref seq1 i) (aref seq2 j)))))
643 seq1))))
645 (macrolet
646 ((define-replace-transforms ()
647 (loop for saetp across sb!vm:*specialized-array-element-type-properties*
648 for sequence-type = `(simple-array ,(sb!vm:saetp-specifier saetp) (*))
649 unless (= (sb!vm:saetp-typecode saetp) sb!vm::simple-array-nil-widetag)
650 collect (make-replace-transform saetp sequence-type sequence-type)
651 into forms
652 finally (return `(progn ,@forms))))
653 (define-one-transform (sequence-type1 sequence-type2)
654 (make-replace-transform nil sequence-type1 sequence-type2)))
655 (define-replace-transforms)
656 (define-one-transform simple-base-string (simple-array character (*)))
657 (define-one-transform (simple-array character (*)) simple-base-string))
659 ;;; Expand simple cases of UB<SIZE>-BASH-COPY inline. "simple" is
660 ;;; defined as those cases where we are doing word-aligned copies from
661 ;;; both the source and the destination and we are copying from the same
662 ;;; offset from both the source and the destination. (The last
663 ;;; condition is there so we can determine the direction to copy at
664 ;;; compile time rather than runtime. Remember that UB<SIZE>-BASH-COPY
665 ;;; acts like memmove, not memcpy.) These conditions may seem rather
666 ;;; restrictive, but they do catch common cases, like allocating a (* 2
667 ;;; N)-size buffer and blitting in the old N-size buffer in.
669 (defun frob-bash-transform (src src-offset
670 dst dst-offset
671 length n-elems-per-word)
672 (declare (ignore src dst length))
673 (let ((n-bits-per-elem (truncate sb!vm:n-word-bits n-elems-per-word)))
674 (multiple-value-bind (src-word src-elt)
675 (truncate (lvar-value src-offset) n-elems-per-word)
676 (multiple-value-bind (dst-word dst-elt)
677 (truncate (lvar-value dst-offset) n-elems-per-word)
678 ;; Avoid non-word aligned copies.
679 (unless (and (zerop src-elt) (zerop dst-elt))
680 (give-up-ir1-transform))
681 ;; Avoid copies where we would have to insert code for
682 ;; determining the direction of copying.
683 (unless (= src-word dst-word)
684 (give-up-ir1-transform))
685 ;; FIXME: The cross-compiler doesn't optimize TRUNCATE properly,
686 ;; so we have to do its work here.
687 `(let ((end (+ ,src-word ,(if (= n-elems-per-word 1)
688 'length
689 `(truncate (the index length) ,n-elems-per-word)))))
690 (declare (type index end))
691 ;; Handle any bits at the end.
692 (when (logtest length (1- ,n-elems-per-word))
693 (let* ((extra (mod length ,n-elems-per-word))
694 ;; FIXME: The shift amount on this ASH is
695 ;; *always* negative, but the backend doesn't
696 ;; have a NEGATIVE-FIXNUM primitive type, so we
697 ;; wind up with a pile of code that tests the
698 ;; sign of the shift count prior to shifting when
699 ;; all we need is a simple negate and shift
700 ;; right. Yuck.
701 (mask (ash #.(1- (ash 1 sb!vm:n-word-bits))
702 (* (- extra ,n-elems-per-word)
703 ,n-bits-per-elem))))
704 (setf (sb!kernel:%vector-raw-bits dst end)
705 (logior
706 (logandc2 (sb!kernel:%vector-raw-bits dst end)
707 (ash mask
708 ,(ecase sb!c:*backend-byte-order*
709 (:little-endian 0)
710 (:big-endian `(* (- ,n-elems-per-word extra)
711 ,n-bits-per-elem)))))
712 (logand (sb!kernel:%vector-raw-bits src end)
713 (ash mask
714 ,(ecase sb!c:*backend-byte-order*
715 (:little-endian 0)
716 (:big-endian `(* (- ,n-elems-per-word extra)
717 ,n-bits-per-elem)))))))))
718 ;; Copy from the end to save a register.
719 (do ((i end (1- i)))
720 ((<= i ,src-word))
721 (setf (sb!kernel:%vector-raw-bits dst (1- i))
722 (sb!kernel:%vector-raw-bits src (1- i)))))))))
724 #.(loop for i = 1 then (* i 2)
725 collect `(deftransform ,(intern (format nil "UB~D-BASH-COPY" i)
726 "SB!KERNEL")
727 ((src src-offset
728 dst dst-offset
729 length)
730 ((simple-unboxed-array (*))
731 (constant-arg index)
732 (simple-unboxed-array (*))
733 (constant-arg index)
734 index)
736 (frob-bash-transform src src-offset
737 dst dst-offset length
738 ,(truncate sb!vm:n-word-bits i))) into forms
739 until (= i sb!vm:n-word-bits)
740 finally (return `(progn ,@forms)))
742 ;;; We expand copy loops inline in SUBSEQ and COPY-SEQ if we're copying
743 ;;; arrays with elements of size >= the word size. We do this because
744 ;;; we know the arrays cannot alias (one was just consed), therefore we
745 ;;; can determine at compile time the direction to copy, and for
746 ;;; word-sized elements, UB<WORD-SIZE>-BASH-COPY will do a bit of
747 ;;; needless checking to figure out what's going on. The same
748 ;;; considerations apply if we are copying elements larger than the word
749 ;;; size, with the additional twist that doing it inline is likely to
750 ;;; cons far less than calling REPLACE and letting generic code do the
751 ;;; work.
753 ;;; However, we do not do this for elements whose size is < than the
754 ;;; word size because we don't want to deal with any alignment issues
755 ;;; inline. The UB*-BASH-COPY transforms might fix things up later
756 ;;; anyway.
758 (defun maybe-expand-copy-loop-inline (src src-offset dst dst-offset length
759 element-type)
760 (let ((saetp (find-saetp element-type)))
761 (aver saetp)
762 (if (>= (sb!vm:saetp-n-bits saetp) sb!vm:n-word-bits)
763 (expand-aref-copy-loop src src-offset dst dst-offset length)
764 `(locally (declare (optimize (safety 0)))
765 (replace ,dst ,src :start1 ,dst-offset :start2 ,src-offset :end1 ,length)))))
767 (defun expand-aref-copy-loop (src src-offset dst dst-offset length)
768 (if (eql src-offset dst-offset)
769 `(do ((i (+ ,src-offset ,length) (1- i)))
770 ((<= i ,src-offset))
771 (declare (optimize (insert-array-bounds-checks 0)))
772 (setf (aref ,dst (1- i)) (aref ,src (1- i))))
773 ;; KLUDGE: The compiler is not able to derive that (+ offset
774 ;; length) must be a fixnum, but arrives at (unsigned-byte 29).
775 ;; We, however, know it must be so, as by this point the bounds
776 ;; have already been checked.
777 `(do ((i (truly-the fixnum (+ ,src-offset ,length)) (1- i))
778 (j (+ ,dst-offset ,length) (1- j)))
779 ((<= i ,src-offset))
780 (declare (optimize (insert-array-bounds-checks 0))
781 (type (integer 0 #.sb!xc:array-dimension-limit) j i))
782 (setf (aref ,dst (1- j)) (aref ,src (1- i))))))
784 (deftransform subseq ((seq start &optional end)
785 ((or (simple-unboxed-array (*)) simple-vector) t &optional t)
786 * :node node)
787 (let ((array-type (lvar-type seq)))
788 (unless (array-type-p array-type)
789 (give-up-ir1-transform))
790 (let ((element-type (type-specifier (array-type-specialized-element-type array-type))))
791 `(let* ((length (length seq))
792 (end (or end length)))
793 ,(unless (policy node (= safety 0))
794 '(progn
795 (unless (<= 0 start end length)
796 (sb!impl::signal-bounding-indices-bad-error seq start end))))
797 (let* ((size (- end start))
798 (result (make-array size :element-type ',element-type)))
799 ,(maybe-expand-copy-loop-inline 'seq (if (constant-lvar-p start)
800 (lvar-value start)
801 'start)
802 'result 0 'size element-type)
803 result)))))
805 (deftransform subseq ((seq start &optional end)
806 (list t &optional t))
807 `(list-subseq* seq start end))
809 (deftransform copy-seq ((seq) ((or (simple-unboxed-array (*)) simple-vector)) *)
810 (let ((array-type (lvar-type seq)))
811 (unless (array-type-p array-type)
812 (give-up-ir1-transform))
813 (let ((element-type (type-specifier (array-type-specialized-element-type array-type))))
814 `(let* ((length (length seq))
815 (result (make-array length :element-type ',element-type)))
816 ,(maybe-expand-copy-loop-inline 'seq 0 'result 0 'length element-type)
817 result))))
819 ;;; FIXME: it really should be possible to take advantage of the
820 ;;; macros used in code/seq.lisp here to avoid duplication of code,
821 ;;; and enable even funkier transformations.
822 (deftransform search ((pattern text &key (start1 0) (start2 0) end1 end2
823 (test #'eql)
824 (key #'identity)
825 from-end)
826 (vector vector &rest t)
828 :policy (> speed (max space safety)))
829 "open code"
830 (let ((from-end (when (lvar-p from-end)
831 (unless (constant-lvar-p from-end)
832 (give-up-ir1-transform ":FROM-END is not constant."))
833 (lvar-value from-end)))
834 (keyp (lvar-p key))
835 (testp (lvar-p test)))
836 `(block search
837 (let ((end1 (or end1 (length pattern)))
838 (end2 (or end2 (length text)))
839 ,@(when keyp
840 '((key (coerce key 'function))))
841 ,@(when testp
842 '((test (coerce test 'function)))))
843 (declare (type index start1 start2 end1 end2))
844 (do (,(if from-end
845 '(index2 (- end2 (- end1 start1)) (1- index2))
846 '(index2 start2 (1+ index2))))
847 (,(if from-end
848 '(< index2 start2)
849 '(>= index2 end2))
850 nil)
851 ;; INDEX2 is FIXNUM, not an INDEX, as right before the loop
852 ;; terminates is hits -1 when :FROM-END is true and :START2
853 ;; is 0.
854 (declare (type fixnum index2))
855 (when (do ((index1 start1 (1+ index1))
856 (index2 index2 (1+ index2)))
857 ((>= index1 end1) t)
858 (declare (type index index1 index2))
859 ,@(unless from-end
860 '((when (= index2 end2)
861 (return-from search nil))))
862 (unless (,@(if testp
863 '(funcall test)
864 '(eql))
865 ,(if keyp
866 '(funcall key (aref pattern index1))
867 '(aref pattern index1))
868 ,(if keyp
869 '(funcall key (aref text index2))
870 '(aref text index2)))
871 (return nil)))
872 (return index2)))))))
875 ;;; Open-code CONCATENATE for strings. It would be possible to extend
876 ;;; this transform to non-strings, but I chose to just do the case that
877 ;;; should cover 95% of CONCATENATE performance complaints for now.
878 ;;; -- JES, 2007-11-17
879 (deftransform concatenate ((result-type &rest lvars)
880 (symbol &rest sequence)
882 :policy (> speed space))
883 (unless (constant-lvar-p result-type)
884 (give-up-ir1-transform))
885 (let* ((element-type (let ((type (lvar-value result-type)))
886 ;; Only handle the simple result type cases. If
887 ;; somebody does (CONCATENATE '(STRING 6) ...)
888 ;; their code won't be optimized, but nobody does
889 ;; that in practice.
890 (case type
891 ((string simple-string) 'character)
892 ((base-string simple-base-string) 'base-char)
893 (t (give-up-ir1-transform)))))
894 (vars (loop for x in lvars collect (gensym)))
895 (lvar-values (loop for lvar in lvars
896 collect (when (constant-lvar-p lvar)
897 (lvar-value lvar))))
898 (lengths
899 (loop for value in lvar-values
900 for var in vars
901 collect (if value
902 (length value)
903 `(sb!impl::string-dispatch ((simple-array * (*))
904 sequence)
905 ,var
906 (declare (muffle-conditions compiler-note))
907 (length ,var))))))
908 `(apply
909 (lambda ,vars
910 (declare (ignorable ,@vars))
911 (let* ((.length. (+ ,@lengths))
912 (.pos. 0)
913 (.string. (make-string .length. :element-type ',element-type)))
914 (declare (type index .length. .pos.)
915 (muffle-conditions compiler-note))
916 ,@(loop for value in lvar-values
917 for var in vars
918 collect (if (stringp value)
919 ;; Fold the array reads for constant arguments
920 `(progn
921 ,@(loop for c across value
922 collect `(setf (aref .string.
923 .pos.) ,c)
924 collect `(incf .pos.)))
925 `(sb!impl::string-dispatch
926 (#!+sb-unicode
927 (simple-array character (*))
928 (simple-array base-char (*))
930 ,var
931 (replace .string. ,var :start1 .pos.)
932 (incf .pos. (length ,var)))))
933 .string.))
934 lvars)))
936 ;;;; CONS accessor DERIVE-TYPE optimizers
938 (defoptimizer (car derive-type) ((cons))
939 (let ((type (lvar-type cons))
940 (null-type (specifier-type 'null)))
941 (cond ((eq type null-type)
942 null-type)
943 ((cons-type-p type)
944 (cons-type-car-type type)))))
946 (defoptimizer (cdr derive-type) ((cons))
947 (let ((type (lvar-type cons))
948 (null-type (specifier-type 'null)))
949 (cond ((eq type null-type)
950 null-type)
951 ((cons-type-p type)
952 (cons-type-cdr-type type)))))
954 ;;;; FIND, POSITION, and their -IF and -IF-NOT variants
956 ;;; We want to make sure that %FIND-POSITION is inline-expanded into
957 ;;; %FIND-POSITION-IF only when %FIND-POSITION-IF has an inline
958 ;;; expansion, so we factor out the condition into this function.
959 (defun check-inlineability-of-find-position-if (sequence from-end)
960 (let ((ctype (lvar-type sequence)))
961 (cond ((csubtypep ctype (specifier-type 'vector))
962 ;; It's not worth trying to inline vector code unless we
963 ;; know a fair amount about it at compile time.
964 (upgraded-element-type-specifier-or-give-up sequence)
965 (unless (constant-lvar-p from-end)
966 (give-up-ir1-transform
967 "FROM-END argument value not known at compile time")))
968 ((csubtypep ctype (specifier-type 'list))
969 ;; Inlining on lists is generally worthwhile.
972 (give-up-ir1-transform
973 "sequence type not known at compile time")))))
975 ;;; %FIND-POSITION-IF and %FIND-POSITION-IF-NOT for LIST data
976 (macrolet ((def (name condition)
977 `(deftransform ,name ((predicate sequence from-end start end key)
978 (function list t t t function)
980 :policy (> speed space))
981 "expand inline"
982 `(let ((index 0)
983 (find nil)
984 (position nil))
985 (declare (type index index))
986 (dolist (i sequence
987 (if (and end (> end index))
988 (sb!impl::signal-bounding-indices-bad-error
989 sequence start end)
990 (values find position)))
991 (let ((key-i (funcall key i)))
992 (when (and end (>= index end))
993 (return (values find position)))
994 (when (>= index start)
995 (,',condition (funcall predicate key-i)
996 ;; This hack of dealing with non-NIL
997 ;; FROM-END for list data by iterating
998 ;; forward through the list and keeping
999 ;; track of the last time we found a match
1000 ;; might be more screwy than what the user
1001 ;; expects, but it seems to be allowed by
1002 ;; the ANSI standard. (And if the user is
1003 ;; screwy enough to ask for FROM-END
1004 ;; behavior on list data, turnabout is
1005 ;; fair play.)
1007 ;; It's also not enormously efficient,
1008 ;; calling PREDICATE and KEY more often
1009 ;; than necessary; but all the
1010 ;; alternatives seem to have their own
1011 ;; efficiency problems.
1012 (if from-end
1013 (setf find i
1014 position index)
1015 (return (values i index))))))
1016 (incf index))))))
1017 (def %find-position-if when)
1018 (def %find-position-if-not unless))
1020 ;;; %FIND-POSITION for LIST data can be expanded into %FIND-POSITION-IF
1021 ;;; without loss of efficiency. (I.e., the optimizer should be able
1022 ;;; to straighten everything out.)
1023 (deftransform %find-position ((item sequence from-end start end key test)
1024 (t list t t t t t)
1026 :policy (> speed space))
1027 "expand inline"
1028 '(%find-position-if (let ((test-fun (%coerce-callable-to-fun test)))
1029 ;; The order of arguments for asymmetric tests
1030 ;; (e.g. #'<, as opposed to order-independent
1031 ;; tests like #'=) is specified in the spec
1032 ;; section 17.2.1 -- the O/Zi stuff there.
1033 (lambda (i)
1034 (funcall test-fun item i)))
1035 sequence
1036 from-end
1037 start
1039 (%coerce-callable-to-fun key)))
1041 ;;; The inline expansions for the VECTOR case are saved as macros so
1042 ;;; that we can share them between the DEFTRANSFORMs and the default
1043 ;;; cases in the DEFUNs. (This isn't needed for the LIST case, because
1044 ;;; the DEFTRANSFORMs for LIST are less choosy about when to expand.)
1045 (defun %find-position-or-find-position-if-vector-expansion (sequence-arg
1046 from-end
1047 start
1048 end-arg
1049 element
1050 done-p-expr)
1051 (with-unique-names (offset block index n-sequence sequence n-end end)
1052 `(let ((,n-sequence ,sequence-arg)
1053 (,n-end ,end-arg))
1054 (with-array-data ((,sequence ,n-sequence :offset-var ,offset)
1055 (,start ,start)
1056 (,end (%check-vector-sequence-bounds
1057 ,n-sequence ,start ,n-end)))
1058 (block ,block
1059 (macrolet ((maybe-return ()
1060 ;; WITH-ARRAY-DATA has already performed bounds
1061 ;; checking, so we can safely elide the checks
1062 ;; in the inner loop.
1063 '(let ((,element (locally (declare (optimize (insert-array-bounds-checks 0)))
1064 (aref ,sequence ,index))))
1065 (when ,done-p-expr
1066 (return-from ,block
1067 (values ,element
1068 (- ,index ,offset)))))))
1069 (if ,from-end
1070 (loop for ,index
1071 ;; (If we aren't fastidious about declaring that
1072 ;; INDEX might be -1, then (FIND 1 #() :FROM-END T)
1073 ;; can send us off into never-never land, since
1074 ;; INDEX is initialized to -1.)
1075 of-type index-or-minus-1
1076 from (1- ,end) downto ,start do
1077 (maybe-return))
1078 (loop for ,index of-type index from ,start below ,end do
1079 (maybe-return))))
1080 (values nil nil))))))
1082 (def!macro %find-position-vector-macro (item sequence
1083 from-end start end key test)
1084 (with-unique-names (element)
1085 (%find-position-or-find-position-if-vector-expansion
1086 sequence
1087 from-end
1088 start
1090 element
1091 ;; (See the LIST transform for a discussion of the correct
1092 ;; argument order, i.e. whether the searched-for ,ITEM goes before
1093 ;; or after the checked sequence element.)
1094 `(funcall ,test ,item (funcall ,key ,element)))))
1096 (def!macro %find-position-if-vector-macro (predicate sequence
1097 from-end start end key)
1098 (with-unique-names (element)
1099 (%find-position-or-find-position-if-vector-expansion
1100 sequence
1101 from-end
1102 start
1104 element
1105 `(funcall ,predicate (funcall ,key ,element)))))
1107 (def!macro %find-position-if-not-vector-macro (predicate sequence
1108 from-end start end key)
1109 (with-unique-names (element)
1110 (%find-position-or-find-position-if-vector-expansion
1111 sequence
1112 from-end
1113 start
1115 element
1116 `(not (funcall ,predicate (funcall ,key ,element))))))
1118 ;;; %FIND-POSITION, %FIND-POSITION-IF and %FIND-POSITION-IF-NOT for
1119 ;;; VECTOR data
1120 (deftransform %find-position-if ((predicate sequence from-end start end key)
1121 (function vector t t t function)
1123 :policy (> speed space))
1124 "expand inline"
1125 (check-inlineability-of-find-position-if sequence from-end)
1126 '(%find-position-if-vector-macro predicate sequence
1127 from-end start end key))
1129 (deftransform %find-position-if-not ((predicate sequence from-end start end key)
1130 (function vector t t t function)
1132 :policy (> speed space))
1133 "expand inline"
1134 (check-inlineability-of-find-position-if sequence from-end)
1135 '(%find-position-if-not-vector-macro predicate sequence
1136 from-end start end key))
1138 (deftransform %find-position ((item sequence from-end start end key test)
1139 (t vector t t t function function)
1141 :policy (> speed space))
1142 "expand inline"
1143 (check-inlineability-of-find-position-if sequence from-end)
1144 '(%find-position-vector-macro item sequence
1145 from-end start end key test))
1147 ;;; logic to unravel :TEST, :TEST-NOT, and :KEY options in FIND,
1148 ;;; POSITION-IF, etc.
1149 (define-source-transform effective-find-position-test (test test-not)
1150 (once-only ((test test)
1151 (test-not test-not))
1152 `(cond
1153 ((and ,test ,test-not)
1154 (error "can't specify both :TEST and :TEST-NOT"))
1155 (,test (%coerce-callable-to-fun ,test))
1156 (,test-not
1157 ;; (Without DYNAMIC-EXTENT, this is potentially horribly
1158 ;; inefficient, but since the TEST-NOT option is deprecated
1159 ;; anyway, we don't care.)
1160 (complement (%coerce-callable-to-fun ,test-not)))
1161 (t #'eql))))
1162 (define-source-transform effective-find-position-key (key)
1163 (once-only ((key key))
1164 `(if ,key
1165 (%coerce-callable-to-fun ,key)
1166 #'identity)))
1168 (macrolet ((define-find-position (fun-name values-index)
1169 `(deftransform ,fun-name ((item sequence &key
1170 from-end (start 0) end
1171 key test test-not)
1172 (t (or list vector) &rest t))
1173 '(nth-value ,values-index
1174 (%find-position item sequence
1175 from-end start
1177 (effective-find-position-key key)
1178 (effective-find-position-test
1179 test test-not))))))
1180 (define-find-position find 0)
1181 (define-find-position position 1))
1183 (macrolet ((define-find-position-if (fun-name values-index)
1184 `(deftransform ,fun-name ((predicate sequence &key
1185 from-end (start 0)
1186 end key)
1187 (t (or list vector) &rest t))
1188 '(nth-value
1189 ,values-index
1190 (%find-position-if (%coerce-callable-to-fun predicate)
1191 sequence from-end
1192 start end
1193 (effective-find-position-key key))))))
1194 (define-find-position-if find-if 0)
1195 (define-find-position-if position-if 1))
1197 ;;; the deprecated functions FIND-IF-NOT and POSITION-IF-NOT. We
1198 ;;; didn't bother to worry about optimizing them, except note that on
1199 ;;; Sat, Oct 06, 2001 at 04:22:38PM +0100, Christophe Rhodes wrote on
1200 ;;; sbcl-devel
1202 ;;; My understanding is that while the :test-not argument is
1203 ;;; deprecated in favour of :test (complement #'foo) because of
1204 ;;; semantic difficulties (what happens if both :test and :test-not
1205 ;;; are supplied, etc) the -if-not variants, while officially
1206 ;;; deprecated, would be undeprecated were X3J13 actually to produce
1207 ;;; a revised standard, as there are perfectly legitimate idiomatic
1208 ;;; reasons for allowing the -if-not versions equal status,
1209 ;;; particularly remove-if-not (== filter).
1211 ;;; This is only an informal understanding, I grant you, but
1212 ;;; perhaps it's worth optimizing the -if-not versions in the same
1213 ;;; way as the others?
1215 ;;; FIXME: Maybe remove uses of these deprecated functions within the
1216 ;;; implementation of SBCL.
1217 (macrolet ((define-find-position-if-not (fun-name values-index)
1218 `(deftransform ,fun-name ((predicate sequence &key
1219 from-end (start 0)
1220 end key)
1221 (t (or list vector) &rest t))
1222 '(nth-value
1223 ,values-index
1224 (%find-position-if-not (%coerce-callable-to-fun predicate)
1225 sequence from-end
1226 start end
1227 (effective-find-position-key key))))))
1228 (define-find-position-if-not find-if-not 0)
1229 (define-find-position-if-not position-if-not 1))