X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fpcl%2Fcompiler-support.lisp;h=846f82a46ac9a12f19225a1d170b32c02730c6f5;hb=b83353d9f998e5c0e34604b5593df70c66d2c510;hp=f32755bf2e951db3e174c223401ed7f1c7c4b15b;hpb=a556288505e2687ac333e1b0a8deebb13c76a60c;p=sbcl.git diff --git a/src/pcl/compiler-support.lisp b/src/pcl/compiler-support.lisp index f32755b..846f82a 100644 --- a/src/pcl/compiler-support.lisp +++ b/src/pcl/compiler-support.lisp @@ -64,9 +64,9 @@ (defvar sb-pcl::*internal-pcl-generalized-fun-name-symbols* nil) -(defmacro define-internal-pcl-function-name-syntax (name &body body) +(defmacro define-internal-pcl-function-name-syntax (name (var) &body body) `(progn - (define-function-name-syntax ,name ,@body) + (define-function-name-syntax ,name (,var) ,@body) (pushnew ',name sb-pcl::*internal-pcl-generalized-fun-name-symbols*))) (define-internal-pcl-function-name-syntax sb-pcl::slot-accessor (list) @@ -84,56 +84,11 @@ (valid-function-name-p (cadr list))) (define-internal-pcl-function-name-syntax sb-pcl::ctor (list) - (valid-function-name-p (cadr list))) - -(defun sb-pcl::random-documentation (name type) - (cdr (assoc type (info :random-documentation :stuff name)))) - -(defun sb-pcl::set-random-documentation (name type new-value) - (let ((pair (assoc type (info :random-documentation :stuff name)))) - (if pair - (setf (cdr pair) new-value) - (push (cons type new-value) - (info :random-documentation :stuff name)))) - new-value) - -(defsetf sb-pcl::random-documentation sb-pcl::set-random-documentation) - -;;;; SLOT-VALUE optimizations - -(defknown slot-value (t symbol) t (any)) -(defknown sb-pcl::set-slot-value (t symbol t) t (any)) - -(defun pcl-boot-state-complete-p () - (eq 'sb-pcl::complete sb-pcl::*boot-state*)) - -;;; These essentially duplicate what the compiler-macros in slots.lisp -;;; do, but catch more cases. We retain the compiler-macros since they -;;; can be used during the build, and because they catch common cases -;;; slightly more cheaply then the transforms. (Transforms add new -;;; lambdas, which requires more work by the compiler.) - -(deftransform slot-value ((object slot-name) * * :important t) - "optimize" - (let (c-slot-name) - (if (and (pcl-boot-state-complete-p) - (constant-lvar-p slot-name) - (setf c-slot-name (lvar-value slot-name)) - (sb-pcl::interned-symbol-p c-slot-name)) - `(sb-pcl::accessor-slot-value object ',c-slot-name) - (give-up-ir1-transform "Slot name is not constant.")))) + (let ((class-or-name (cadr list))) + (cond + ((symbolp class-or-name) + (values (valid-function-name-p class-or-name) nil)) + ((or (sb-pcl::std-instance-p class-or-name) + (sb-pcl::fsc-instance-p class-or-name)) + (values t nil))))) -(deftransform sb-pcl::set-slot-value ((object slot-name new-value) - (t symbol t) t - :important t - ;; see comment in the - ;; compiler-macro - :policy (< safety 3)) - "optimize" - (let (c-slot-name) - (if (and (pcl-boot-state-complete-p) - (constant-lvar-p slot-name) - (setf c-slot-name (lvar-value slot-name)) - (sb-pcl::interned-symbol-p c-slot-name)) - `(sb-pcl::accessor-set-slot-value object ',c-slot-name new-value) - (give-up-ir1-transform "Slot name is not constant."))))