diff options
| author | ringabout <43030857+ringabout@users.noreply.github.com> | 2023-11-28 22:11:43 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-11-28 15:11:43 +0100 |
| commit | 30cf33f04dfb768182accb3ad35ec6364c998664 (patch) | |
| tree | 63dc0d6a36bc662b94246392e03a26b7eaeb116d /lib/system/arc.nim | |
| parent | 8cad6ac0487800f7d41e38303e52129777e6c22e (diff) | |
| download | Nim-30cf33f04dfb768182accb3ad35ec6364c998664.tar.gz | |
rework the vtable implementation embedding the vtable array directly with new strictions on methods (#22991)
**TODO** - [x] fixes changelog With the new option `nimPreviewVtables`, `methods` are confined in the same module where the type of the first parameter is defined - [x] make it opt in after CI checks its feasibility ## In the following-up PRs - [ ] in the following PRs, refactor code into a more efficient one - [ ] cpp needs special treatments since it cannot embed array in light of the preceding limits: ref https://github.com/nim-lang/Nim/pull/20977#discussion_r1035528927; we can support cpp backends with vtable implementations later on the comprise that uses indirect vtable access --------- Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
Diffstat (limited to 'lib/system/arc.nim')
| -rw-r--r-- | lib/system/arc.nim | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/system/arc.nim b/lib/system/arc.nim index b49a6a63b..88d21643a 100644 --- a/lib/system/arc.nim +++ b/lib/system/arc.nim @@ -243,3 +243,8 @@ template tearDownForeignThreadGc* = proc isObjDisplayCheck(source: PNimTypeV2, targetDepth: int16, token: uint32): bool {.compilerRtl, inl.} = result = targetDepth <= source.depth and source.display[targetDepth] == token + +when defined(nimPreviewVtables) and not defined(cpp): + proc nimGetVTable(p: pointer, index: int): pointer + {.compilerRtl, inline, raises: [].} = + result = cast[ptr PNimTypeV2](p).vTable[index] |