Skip to content

Commit

Permalink
boundscheck to inbounds in comprehension lowering.
Browse files Browse the repository at this point in the history
  • Loading branch information
blakejohnson committed Jan 14, 2016
1 parent 03a4e51 commit b74533a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/julia-syntax.scm
Original file line number Diff line number Diff line change
Expand Up @@ -1878,9 +1878,9 @@
(if (null? ranges)
`(block (= ,oneresult ,expr)
,@(if atype '() `((type_goto ,initlabl ,oneresult)))
(boundscheck false)
(inbounds false)
(call (top setindex!) ,result ,oneresult ,ri)
(boundscheck pop)
(inbounds pop)
(= ,ri (call (top +) ,ri 1)))
`(block
(= ,(car states) (call (top start) ,(car rv)))
Expand Down

2 comments on commit b74533a

@yuyichao
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, is the logic reverted here?

@blakejohnson
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yuyichao Good catch, it should be (inbounds true) at the top.

Please sign in to comment.