Skip to content
GitLab
Explore
Sign in
Register
Primary navigation
Search or go to…
Project
cmucl
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Package Registry
Model registry
Operate
Environments
Terraform modules
Monitor
Incidents
Service Desk
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Terms and privacy
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Carl Shapiro
cmucl
Commits
38741342
Commit
38741342
authored
33 years ago
by
ram
Browse files
Options
Downloads
Patches
Plain Diff
Fixed a bug that caused an internal error when a never-referenced function
(e.g. from FLET) had non-local exit code in it.
parent
6bf1662c
No related branches found
Branches containing commit
No related tags found
Tags containing commit
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
compiler/dfo.lisp
+17
-16
17 additions, 16 deletions
compiler/dfo.lisp
with
17 additions
and
16 deletions
compiler/dfo.lisp
+
17
−
16
View file @
38741342
...
...
@@ -7,7 +7,7 @@
;;; Scott Fahlman or slisp-group@cs.cmu.edu.
;;;
(
ext:file-comment
"$Header: /Volumes/share2/src/cmucl/cvs2git/cvsroot/src/compiler/dfo.lisp,v 1.
19
199
1/1
2/1
1
1
7:58:06
ram Exp $"
)
"$Header: /Volumes/share2/src/cmucl/cvs2git/cvsroot/src/compiler/dfo.lisp,v 1.
20
199
2/0
2/1
2
1
9:20:12
ram Exp $"
)
;;;
;;; **********************************************************************
;;;
...
...
@@ -115,11 +115,6 @@
;;; Walk-Home-Call-Graph -- Internal
;;;
;;; This function ensures that all the blocks in a given environment will be
;;; in the same component, even when they might not seem reachable from the
;;; environment entry. Consider the case of code that is only reachable from a
;;; non-local exit.
;;;
;;; This function is called on each block by Find-Initial-DFO-Aux before it
;;; walks the successors. It looks at the home lambda's bind block to see if
;;; that block is in some other component:
...
...
@@ -127,6 +122,10 @@
;;; the home function to move it into component.
;;; -- If the block is in some other component, join Component into it and
;;; return that component.
;;; -- If the home function is deleted, do nothing. Block must eventually be
;;; discovered to be unreachable as well. This can happen when we have a
;;; NLX into a function with no references. The escape function still has
;;; refs (in the deleted function).
;;;
;;; This ensures that all the blocks in a given environment will be in the same
;;; component, even when they might not seem reachable from the environment
...
...
@@ -135,16 +134,18 @@
;;;
(
defun
walk-home-call-graph
(
block
component
)
(
declare
(
type
cblock
block
)
(
type
component
component
))
(
let*
((
home
(
block-home-lambda
block
))
(
bind-block
(
node-block
(
lambda-bind
home
)))
(
home-component
(
block-component
bind-block
)))
(
cond
((
eq
(
component-kind
home-component
)
:initial
)
(
dfo-walk-call-graph
home
component
))
((
eq
home-component
component
)
component
)
(
t
(
join-components
home-component
component
)
home-component
))))
(
let
((
home
(
block-home-lambda
block
)))
(
if
(
eq
(
functional-kind
home
)
:deleted
)
component
(
let*
((
bind-block
(
node-block
(
lambda-bind
home
)))
(
home-component
(
block-component
bind-block
)))
(
cond
((
eq
(
component-kind
home-component
)
:initial
)
(
dfo-walk-call-graph
home
component
))
((
eq
home-component
component
)
component
)
(
t
(
join-components
home-component
component
)
home-component
))))))
;;; Find-Initial-DFO-Aux -- Internal
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment