Skip to content

more IC related refactorings #25276

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 55 commits into
base: devel
Choose a base branch
from
Open

more IC related refactorings #25276

wants to merge 55 commits into from
+5,853 −1,625

Conversation

@Araq
Copy link
Member

@Araq Araq commented Nov 11, 2025 •
edited
Loading

TODO:

  • test writing of .nif files
  • implement loading of fields in PType/PSym that might not have been loaded
  • implement interface logic
  • implement pragma "replays"
  • implement special logic for converter
  • implement special logic for method
  • test the logic holds up for export
  • implement logic to free the memory of PSym/PType if memory pressure is high
  • implement logic to close memory mapped files if too many are open.

demotomohiro and others added 30 commits November 6, 2025 04:50
@demotomohiro
adds Nim's AST to/from NIF
e9127b2
@demotomohiro
encodes/decodes itemId.module
25ca7cc
@demotomohiro
uses PSym.item and PType.item as id in NIF
5eeede3
@demotomohiro
saves/loads PSym guard, bitsize and alignment
51eccea
@demotomohiro
saves/loads PNode and PSym line info
8a4d921
@demotomohiro
saves/loads symbol magic, options and offset
6f237be
@demotomohiro
adds procedure test code and fix bugs
b50c453
@demotomohiro
adds more procedure test code
3ceb223
@demotomohiro
adds statements test code
10885d4
@demotomohiro
fixes assertion defect when encoding empty object type definition
6a74316
@demotomohiro
adds pragma test code
467a279
@demotomohiro
adds generics test code and fixes bug
66d7a8e
@demotomohiro
adds expression test code
3c599ed
@demotomohiro
uses ItemId instead of PSym or PType for the set of decoded syms and …
91db9ac 
…types
@demotomohiro
saves/loads float Inf, NaN and NegInf literals
9f73412
@demotomohiro
saves/loads flags of nkIdent node
f3178e4
@demotomohiro
makes tencode_node2node.nim faster
5f3be1e
@demotomohiro
saves/loads PSym.ast, constraint and instantiatedFrom
ab13900
@demotomohiro
refers imported symbols like Nimony
54aa552
@demotomohiro
loads modules imported by system module and create module sym before …
43ff7b1 
…decode
@ringabout @Araq
fixes #25263; provides a new switch mangle:nim/cpp for debug name m…
116c220 
…angling (#25264)

fixes #25263

- [x] documentation and changelogs
@arnetheduck @Araq
Add heaptrack support (#25257)
155b06a 
This PR, courtesy of @NagyZoltanPeter
(waku-org/nwaku#3522) adds the ability to track
memory allocations in a program suitable for use with
[heaptrack](https://github.com/KDE/heaptrack).

By passing `-d:heaptrack --debugger:native` to compilation, calls to
heaptrack will be injected when memory is being allocated and released -
unlike `-d:useMalloc` this strategy also works with `refc` and the
default memory pool.

See https://github.com/KDE/heaptrack for usage examples. The resulting
binary needs to be run with `heaptrack` and with the shared
`libheaptrack_preload.so` in the `LD_LIBRARY_PATH`.
@ringabout @Araq
fixes #25251; SIGBUS with iterator over const Table lookup - prematur…
6d3254a 
…e temporary destruction (#25255)

fixes #25251

enforce a copy if the arg is a deref of a lent pointer since the arg
could be a temporary that will go out of scope
@Araq
VM: optimize 'return' slots; saves millions of node allocations for N… (
a866bf2 
#25266)

…imbus
@Araq
wip
6a2f4d2
@Araq
Merge branch 'devel' into demotomohiro-add-ast2nif2
dc2fb81
@Araq
wip
b4c942c
@Araq
progress
e6a1c14
@Araq
prepare for dependency tracking
940ce87
@Araq
progress
0bf8bbf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Araq @demotomohiro @ringabout @arnetheduck