Hoist tests from scan_weak_pointers() into scav_weak_pointer()
[sbcl.git] / src / compiler / checkgen.lisp
blob04584a614f3a49ba5e68b1f7da818dbd820f78f5
1 ;;;; This file implements type check generation. This is a phase that
2 ;;;; runs at the very end of IR1. If a type check is too complex for
3 ;;;; the back end to directly emit in-line, then we transform the check
4 ;;;; into an explicit conditional using TYPEP.
6 ;;;; This software is part of the SBCL system. See the README file for
7 ;;;; more information.
8 ;;;;
9 ;;;; This software is derived from the CMU CL system, which was
10 ;;;; written at Carnegie Mellon University and released into the
11 ;;;; public domain. The software is in the public domain and is
12 ;;;; provided with absolutely no warranty. See the COPYING and CREDITS
13 ;;;; files for more information.
15 (in-package "SB!C")
17 ;;;; cost estimation
19 ;;; Return some sort of guess about the cost of a call to a function.
20 ;;; If the function has some templates, we return the cost of the
21 ;;; cheapest one, otherwise we return the cost of CALL-NAMED. Calling
22 ;;; this with functions that have transforms can result in relatively
23 ;;; meaningless results (exaggerated costs.)
24 ;;;
25 ;;; We special-case NULL, since it does have a source tranform and is
26 ;;; interesting to us.
27 (defun fun-guessed-cost (name)
28 (declare (symbol name))
29 (let ((info (info :function :info name))
30 (call-cost (template-cost (template-or-lose 'call-named))))
31 (if info
32 (let ((templates (fun-info-templates info)))
33 (if templates
34 (template-cost (first templates))
35 (case name
36 (null (template-cost (template-or-lose 'if-eq)))
37 (t call-cost))))
38 call-cost)))
40 ;;; Return some sort of guess for the cost of doing a test against
41 ;;; TYPE. The result need not be precise as long as it isn't way out
42 ;;; in space. The units are based on the costs specified for various
43 ;;; templates in the VM definition.
44 (defun type-test-cost (type)
45 (declare (type ctype type))
46 (or (when (eq type *universal-type*)
48 (when (eq type *empty-type*)
50 (let ((found (cdr (assoc type *backend-type-predicates*
51 :test #'type=))))
52 (if found
53 (+ (fun-guessed-cost found) (fun-guessed-cost 'eq))
54 nil))
55 (typecase type
56 (compound-type
57 (reduce #'+ (compound-type-types type) :key 'type-test-cost))
58 (member-type
59 (* (member-type-size type)
60 (fun-guessed-cost 'eq)))
61 (numeric-type
62 (* (if (numeric-type-complexp type) 2 1)
63 (fun-guessed-cost
64 (if (csubtypep type (specifier-type 'fixnum)) 'fixnump 'numberp))
65 (+ 1
66 (if (numeric-type-low type) 1 0)
67 (if (numeric-type-high type) 1 0))))
68 (cons-type
69 (+ (type-test-cost (specifier-type 'cons))
70 (fun-guessed-cost 'car)
71 (type-test-cost (cons-type-car-type type))
72 (fun-guessed-cost 'cdr)
73 (type-test-cost (cons-type-cdr-type type))))
75 (fun-guessed-cost 'typep)))))
77 (defun weaken-integer-type (type &key range-only)
78 ;; FIXME: Our canonicalization isn't quite ideal for this. We get
79 ;; types such as:
81 ;; (OR (AND (SATISFIES FOO) (INTEGER -100 -50))
82 ;; (AND (SATISFIES FOO) (INTEGER 100 200)))
84 ;; here, and weakening that into
86 ;; (AND (SATISFIES FOO) (INTEGER -100 200))
88 ;; is too much work to do here ... but if we canonicalized things
89 ;; differently, we could get it for free with trivial changes here.
90 (labels ((weaken-integer-type-part (type base)
91 (cond ((intersection-type-p type)
92 (let ((new (specifier-type base)))
93 (dolist (part (intersection-type-types type))
94 (when (if range-only
95 (numeric-type-p part)
96 (not (unknown-type-p part)))
97 (setf new (type-intersection
98 new (weaken-integer-type-part part t)))))
99 new))
100 ((union-type-p type)
101 (let ((low t) (high t) (rest *empty-type*))
102 (flet ((maximize (bound)
103 (if (and bound high)
104 (setf high (if (eq t high)
105 bound
106 (max high bound)))
107 (setf high nil)))
108 (minimize (bound)
109 (if (and bound low)
110 (setf low (if (eq t low)
111 bound
112 (min low bound)))
113 (setf low nil))))
114 (dolist (part (union-type-types type))
115 (let ((weak (weaken-integer-type-part part t)))
116 (cond ((numeric-type-p weak)
117 (minimize (numeric-type-low weak))
118 (maximize (numeric-type-high weak)))
119 ((not range-only)
120 (setf rest (type-union rest weak)))))))
121 (if (eq t low)
122 rest
123 (type-union rest
124 (specifier-type
125 `(integer ,(or low '*) ,(or high '*)))))))
127 type))))
128 (weaken-integer-type-part type 'integer)))
130 (defun-cached
131 (weaken-type :hash-bits 7 :hash-function #'type-hash-value)
132 ((type eq))
133 (declare (type ctype type))
134 (cond ((named-type-p type)
135 type)
136 ((csubtypep type (specifier-type 'integer))
137 ;; Simple range checks are not that expensive, and we *don't*
138 ;; want to accidentally lose eg. array bounds checks due to
139 ;; weakening, so for integer types we simply collapse all
140 ;; ranges into one.
141 (weaken-integer-type type))
143 (let ((min-cost (type-test-cost type))
144 (min-type type)
145 (found-super nil))
146 (dolist (x *backend-type-predicates*)
147 (let* ((stype (car x))
148 (samep (type= stype type)))
149 (when (or samep
150 (and (csubtypep type stype)
151 (not (union-type-p stype))))
152 (let ((stype-cost (type-test-cost stype)))
153 (when (or (< stype-cost min-cost)
154 samep)
155 ;; If the supertype is equal in cost to the type, we
156 ;; prefer the supertype. This produces a closer
157 ;; approximation of the right thing in the presence of
158 ;; poor cost info.
159 (setq found-super t
160 min-type stype
161 min-cost stype-cost))))))
162 ;; This used to return the *UNIVERSAL-TYPE* if no supertype was found,
163 ;; but that's too liberal: it's far too easy for the user to create
164 ;; a union type (which are excluded above), and then trick the compiler
165 ;; into trusting the union type... and finally ending up corrupting the
166 ;; heap once a bad object sneaks past the missing type check.
167 (if found-super
168 min-type
169 type)))))
171 (defun weaken-values-type (type)
172 (declare (type ctype type))
173 (cond ((eq type *wild-type*) type)
174 ((not (values-type-p type))
175 (weaken-type type))
177 (make-values-type :required (mapcar #'weaken-type
178 (values-type-required type))
179 :optional (mapcar #'weaken-type
180 (values-type-optional type))
181 :rest (acond ((values-type-rest type)
182 (weaken-type it)))))))
184 ;;;; checking strategy determination
186 ;;; Return the type we should test for when we really want to check
187 ;;; for TYPE. If type checking policy is "fast", then we return a
188 ;;; weaker type if it is easier to check. First we try the defined
189 ;;; type weakenings, then look for any predicate that is cheaper.
190 (defun maybe-weaken-check (type policy)
191 (declare (type ctype type))
192 (ecase (policy policy type-check)
193 (0 *wild-type*)
194 (2 (weaken-values-type type))
195 (3 type)))
197 ;;; LVAR is an lvar we are doing a type check on and TYPES is a list
198 ;;; of types that we are checking its values against. If we have
199 ;;; proven that LVAR generates a fixed number of values, then for each
200 ;;; value, we check whether it is cheaper to then difference between
201 ;;; the proven type and the corresponding type in TYPES.
202 (defun maybe-negate-check (lvar types original-types n-required)
203 (declare (type lvar lvar) (list types original-types))
204 (let ((ptypes (values-type-out (lvar-derived-type lvar) (length types))))
205 (loop for p in ptypes
206 and c in types
207 and a in original-types
208 and i from 0
209 for cc = (if (>= i n-required)
210 (type-union c (specifier-type 'null))
212 for diff = (type-difference p cc)
213 collect (if (and diff
214 (< (type-test-cost diff)
215 (type-test-cost cc)))
216 (list t diff a)
217 (list nil cc a)))))
219 ;;; Determine if CAST can be checked.
220 ;;; We may check only fixed number of values; in any case the number
221 ;;; of generated values is trusted. If we know the number of produced
222 ;;; values, all of them are checked; otherwise if we know the number
223 ;;; of consumed -- only they are checked; otherwise the check is not
224 ;;; performed.
226 ;;; In the types are checkable it returns :SIMPLE and the second value
227 ;;; of the form:
228 ;;; (NOT-P TYPE ORIGINAL-TYPE)
230 ;;; If true, the NOT-P flag indicates a test that the corresponding
231 ;;; value is *not* of the specified TYPE. ORIGINAL-TYPE is the type
232 ;;; asserted on this value in the lvar, for use in error
233 ;;; messages. When NOT-P is true, this will be different from TYPE.
235 ;;; This allows us to take what has been proven about CAST's argument
236 ;;; type into consideration. If it is cheaper to test for the
237 ;;; difference between the derived type and the asserted type, then we
238 ;;; check for the negation of this type instead.
239 (defun cast-check-types (cast)
240 (declare (type cast cast))
241 (let* ((ctype (coerce-to-values (cast-type-to-check cast)))
242 (atype (coerce-to-values (cast-asserted-type cast)))
243 (dtype (node-derived-type cast))
244 (value (cast-value cast))
245 (lvar (node-lvar cast))
246 (dest (and lvar (lvar-dest lvar)))
247 (n-consumed (cond ((not lvar)
248 nil)
249 ((lvar-single-value-p lvar)
251 ((and (mv-combination-p dest)
252 (eq (mv-combination-kind dest) :local)
253 (lvar-uses (mv-combination-fun dest))
254 (singleton-p (mv-combination-args dest)))
255 (let ((fun-ref (lvar-use (mv-combination-fun dest))))
256 (length (lambda-vars (ref-leaf fun-ref)))))))
257 (n-required (length (values-type-required dtype))))
258 (aver (not (eq ctype *wild-type*)))
259 (cond ((and (null (values-type-optional dtype))
260 (not (values-type-rest dtype)))
261 ;; we [almost] know how many values are produced
262 (values :simple
263 (maybe-negate-check value
264 (values-type-out ctype n-required)
265 (values-type-out atype n-required)
266 n-required)))
267 ((lvar-single-value-p lvar)
268 ;; exactly one value is consumed
269 (principal-lvar-single-valuify lvar)
270 (values :simple (maybe-negate-check value
271 (list (single-value-type ctype))
272 (list (single-value-type atype))
273 n-required)))
274 ((and (mv-combination-p dest)
275 (eq (mv-combination-kind dest) :local)
276 (singleton-p (mv-combination-args dest)))
277 ;; we know the number of consumed values
278 (values :simple (maybe-negate-check value
279 (adjust-list (values-type-types ctype)
280 n-consumed
281 *universal-type*)
282 (adjust-list (values-type-types atype)
283 n-consumed
284 *universal-type*)
285 n-required)))
287 (values :too-hairy nil)))))
289 ;;; Return T is the cast appears to be from the declaration of the callee,
290 ;;; and should be checked externally -- that is, by the callee and not the caller.
291 (defun cast-externally-checkable-p (cast)
292 (declare (type cast cast))
293 (let* ((lvar (node-lvar cast))
294 (dest (and lvar (lvar-dest lvar))))
295 (and (combination-p dest)
296 ;; The theory is that the type assertion is from a declaration on the
297 ;; callee, so the callee should be able to do the check. We want to
298 ;; let the callee do the check, because it is possible that by the
299 ;; time of call that declaration will be changed and we do not want
300 ;; to make people recompile all calls to a function when they were
301 ;; originally compiled with a bad declaration.
303 ;; ALMOST-IMMEDIATELY-USED-P ensures that we don't delegate casts
304 ;; that occur before nodes that can cause observable side effects --
305 ;; most commonly other non-external casts: so the order in which
306 ;; possible type errors are signalled matches with the evaluation
307 ;; order.
309 ;; FIXME: We should let more cases be handled by the callee then we
310 ;; currently do, see: https://bugs.launchpad.net/sbcl/+bug/309104
311 ;; This is not fixable quite here, though, because flow-analysis has
312 ;; deleted the LVAR of the cast by the time we get here, so there is
313 ;; no destination. Perhaps we should mark cases inserted by
314 ;; ASSERT-CALL-TYPE explicitly, and delete those whose destination is
315 ;; deemed unreachable?
316 (almost-immediately-used-p lvar cast)
317 (values (values-subtypep (lvar-externally-checkable-type lvar)
318 (cast-type-to-check cast))))))
320 ;; Type specifiers handled by the general-purpose MAKE-TYPE-CHECK-FORM are often
321 ;; trivial enough to have an internal error number assigned to them that can be
322 ;; used in lieu of OBJECT-NOT-TYPE-ERROR. On x86-64 this saves 16 bytes: 1 word
323 ;; for the symbol in the function's constant area, a MOV instruction to load it,
324 ;; and an sc-offset in the error trap.
325 (defglobal **type-spec-interr-symbols**
326 (let* ((entries
327 ;; read-time-eval so that during cold-init we can recreate the
328 ;; table using the target's sxhash function, but without relying
329 ;; on readiness of the type system for parsing/unparsing specifiers.
330 #.(map 'vector
331 (lambda (entry)
332 (cons (type-specifier (specifier-type (car entry)))
333 (cdr entry)))
334 (remove-if #'stringp sb!c:+backend-internal-errors+
335 :key #'car)))
336 ;; This is effectively a compact read-only binned hashtable.
337 (hashtable (make-array (logior (length entries) 1)
338 :initial-element nil)))
339 (map nil
340 (lambda (entry)
341 (let* ((canon-type (car entry))
342 (bucket (mod (sxhash canon-type) (length hashtable))))
343 (push entry (svref hashtable bucket))))
344 entries)
345 hashtable))
346 (defun %interr-symbol-for-type-spec (spec)
347 (let ((table **type-spec-interr-symbols**))
348 (cadr (assoc spec (svref table (rem (sxhash spec) (length table)))
349 :test #'equal))))
350 #+nil ; some meta-analysis to decide what types should be in "generic/interr"
351 (progn
352 (defvar *checkgen-used-types* (make-hash-table :test 'equal))
353 (defun interr-symbol-for-type-spec (spec)
354 (let ((answer (%interr-symbol-for-type-spec spec))
355 (meta (gethash spec *checkgen-used-types*)))
356 ;; spec -> (count . primitive-p)
357 (if meta
358 (incf (car meta))
359 (setf (gethash spec *checkgen-used-types*) (cons 1 answer)))
360 answer)))
362 (defun internal-type-error-call (var type &optional context)
363 (let* ((external-spec (if (ctype-p type)
364 (type-specifier type)
365 type))
366 (interr-symbol
367 (%interr-symbol-for-type-spec external-spec)))
368 (if interr-symbol
369 `(%type-check-error/c ,var ',interr-symbol ',context)
370 `(%type-check-error ,var ',external-spec ',context))))
372 ;;; Return a lambda form that we can convert to do a type check
373 ;;; of the specified TYPES. TYPES is a list of the format returned by
374 ;;; CAST-CHECK-TYPES.
376 ;;; Note that we don't attempt to check for required values being
377 ;;; unsupplied. Such checking is impossible to efficiently do at the
378 ;;; source level because our fixed-values conventions are optimized
379 ;;; for the common MV-BIND case.
380 (defun make-type-check-form (types &optional context)
381 (let ((temps (make-gensym-list (length types))))
382 `(multiple-value-bind ,temps 'dummy
383 ,@(mapcar (lambda (temp type)
384 (let* ((spec
385 (let ((*unparse-fun-type-simplify* t))
386 (type-specifier (second type))))
387 (test (if (first type) `(not ,spec) spec)))
388 `(unless (typep ,temp ',test)
389 ,(internal-type-error-call temp (third type) context))))
390 temps
391 types)
392 (values ,@temps))))
394 ;;; Splice in explicit type check code immediately before CAST. This
395 ;;; code receives the value(s) that were being passed to CAST-VALUE,
396 ;;; checks the type(s) of the value(s), then passes them further.
397 (defun convert-type-check (cast types)
398 (declare (type cast cast) (type list types))
399 (let ((value (cast-value cast))
400 (length (length types)))
401 (filter-lvar value (make-type-check-form types
402 (cast-context cast)))
403 (reoptimize-lvar (cast-value cast))
404 (setf (cast-type-to-check cast) *wild-type*)
405 (setf (cast-%type-check cast) nil)
406 (let* ((atype (cast-asserted-type cast))
407 (atype (cond ((not (values-type-p atype))
408 atype)
409 ((= length 1)
410 (single-value-type atype))
412 (make-values-type
413 :required (values-type-in atype length)))))
414 (dtype (node-derived-type cast))
415 (dtype (make-values-type
416 :required (values-type-in dtype length))))
417 (setf (cast-asserted-type cast) atype)
418 (setf (node-derived-type cast) dtype)))
420 (values))
422 ;;; Check all possible arguments of CAST and emit type warnings for
423 ;;; those with type errors. If the value of USE is being used for a
424 ;;; variable binding, we figure out which one for source context. If
425 ;;; the value is a constant, we print it specially.
426 (defun cast-check-uses (cast)
427 (declare (type cast cast))
428 (let* ((lvar (node-lvar cast))
429 (dest (and lvar (lvar-dest lvar)))
430 (value (cast-value cast))
431 (atype (cast-asserted-type cast))
432 (condition 'type-warning)
433 (not-ok-uses '()))
434 (do-uses (use value)
435 (let ((dtype (node-derived-type use)))
436 (if (values-types-equal-or-intersect dtype atype)
437 (setf condition 'type-style-warning)
438 (push use not-ok-uses))))
439 (dolist (use (nreverse not-ok-uses))
440 (let* ((*compiler-error-context* use)
441 (dtype (node-derived-type use))
442 (what (when (and (combination-p dest)
443 (eq (combination-kind dest) :local))
444 (let ((lambda (combination-lambda dest))
445 (pos (position-or-lose
446 lvar (combination-args dest))))
447 (format nil "~:[A possible~;The~] binding of ~S"
448 (and (lvar-has-single-use-p lvar)
449 (eq (functional-kind lambda) :let))
450 (leaf-source-name (elt (lambda-vars lambda)
451 pos)))))))
452 (cond ((and (ref-p use) (constant-p (ref-leaf use)))
453 (warn condition
454 :format-control "~:[This~;~:*~A~] is not a ~
455 ~<~%~9T~:;~/sb!impl:print-type/:~>~% ~S"
456 :format-arguments
457 (list what atype (constant-value (ref-leaf use)))))
459 (warn condition
460 :format-control
461 "~:[Result~;~:*~A~] is a ~/sb!impl:print-type/, ~
462 ~<~%~9T~:;not a ~/sb!impl:print-type/.~>"
463 :format-arguments (list what dtype atype)))))))
464 (values))
466 ;;; Loop over all blocks in COMPONENT that have TYPE-CHECK set,
467 ;;; looking for CASTs with TYPE-CHECK T. We do two mostly unrelated
468 ;;; things: detect compile-time type errors and determine if and how
469 ;;; to do run-time type checks.
471 ;;; If there is a compile-time type error, then we mark the CAST and
472 ;;; emit a warning if appropriate. This part loops over all the uses
473 ;;; of the continuation, since after we convert the check, the
474 ;;; :DELETED kind will inhibit warnings about the types of other uses.
476 ;;; If the cast is too complex to be checked by the back end, or is
477 ;;; better checked with explicit code, then convert to an explicit
478 ;;; test. Assertions that can checked by the back end are passed
479 ;;; through. Assertions that can't be tested are flamed about and
480 ;;; marked as not needing to be checked.
482 ;;; If we determine that a type check won't be done, then we set
483 ;;; TYPE-CHECK to :NO-CHECK. In the non-hairy cases, this is just to
484 ;;; prevent us from wasting time coming to the same conclusion again
485 ;;; on a later iteration. In the hairy case, we must indicate to LTN
486 ;;; that it must choose a safe implementation, since IR2 conversion
487 ;;; will choke on the check.
489 ;;; The generation of the type checks is delayed until all the type
490 ;;; check decisions have been made because the generation of the type
491 ;;; checks creates new nodes whose derived types aren't always updated
492 ;;; which may lead to inappropriate template choices due to the
493 ;;; modification of argument types.
494 (defun generate-type-checks (component)
495 (collect ((casts))
496 (do-blocks (block component)
497 (when (and (block-type-check block)
498 (not (block-delete-p block)))
499 ;; CAST-EXTERNALLY-CHECKABLE-P wants the backward pass
500 (do-nodes-backwards (node nil block)
501 (when (and (cast-p node)
502 (cast-type-check node))
503 (cast-check-uses node)
504 (cond ((cast-externally-checkable-p node)
505 (setf (cast-%type-check node) :external))
507 ;; it is possible that NODE was marked :EXTERNAL by
508 ;; the previous pass
509 (setf (cast-%type-check node) t)
510 (casts node)))))
511 (setf (block-type-check block) nil)))
512 (dolist (cast (casts))
513 (unless (bound-cast-p cast)
514 (multiple-value-bind (check types) (cast-check-types cast)
515 (ecase check
516 (:simple
517 (convert-type-check cast types))
518 (:too-hairy
519 (let ((*compiler-error-context* cast))
520 (when (policy cast (>= safety inhibit-warnings))
521 (compiler-notify
522 "type assertion too complex to check:~%~
523 ~/sb!impl:print-type/."
524 (coerce-to-values (cast-asserted-type cast)))))
525 (setf (cast-type-to-check cast) *wild-type*)
526 (setf (cast-%type-check cast) nil)))))))
527 (values))