Skip to content

Commit

Permalink
fixes #24630; static openArray backed by seq cannot be passed to anot…
Browse files Browse the repository at this point in the history
…her function (#24638)

fixes #24630

(cherry picked from commit 2f402fc)
  • Loading branch information
ringabout authored and narimiran committed Jan 24, 2025
1 parent 772ebe6 commit e2ae7d5
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
2 changes: 1 addition & 1 deletion compiler/vmgen.nim
Original file line number Diff line number Diff line change
Expand Up @@ -2013,7 +2013,7 @@ proc genArrayConstr(c: PCtx, n: PNode, dest: var TDest) =
c.gABx(n, opcLdNull, dest, c.genType(n.typ))

let intType = getSysType(c.graph, n.info, tyInt)
let seqType = n.typ.skipTypes(abstractVar-{tyTypeDesc})
let seqType = n.typ.skipTypes(abstractVar+{tyStatic}-{tyTypeDesc})
if seqType.kind == tySequence:
var tmp = c.getTemp(intType)
c.gABx(n, opcLdImmInt, tmp, n.len)
Expand Down
9 changes: 9 additions & 0 deletions tests/vm/topenarrays.nim
Original file line number Diff line number Diff line change
Expand Up @@ -67,3 +67,12 @@ template fn=
doAssert test([0,1,2,3,4,5]).id == 0
fn() # ok
static: fn()

block: # bug #24630
func f(a: static openArray[int]): int =
12

func g(a: static openArray[int]) =
const b = f(a)

g(@[1,2,3])

0 comments on commit e2ae7d5

Please sign in to comment.