julia
Improve `reinterpret` performance for padded types, with minimal harm to compilation time
#60415
Open
Open
Improve `reinterpret` performance for padded types, with minimal harm to compilation time #60415
NHDaly
wants to merge
41 commits into
master
from
nhd/reinterpret-padded-struct-performance
Import fast_reinterpret directly from RAI
7cab3027
disable fallbacks that maybe aren't needed?
5854df18
First attempt at recursive padded-padded src=>dst
c8915e6a
Constant-folding version, which only uses tuple-unrolling
fa0edf8a
Trying out performance tweaks; nothing really helped
32f75970
Revert "Trying out performance tweaks; nothing really helped"
4d0ff614
FINALLY fixed the terribly compilation time.
089cb42a
Simplify the iteration in match_packed_regions
d0d57dfd
Way faster compilation. :) Use depth-first iteration instead of recur…
bb11f4ed
Reuse existing Base.padding() for even faster compilation!!!
e39093f3
Optimization for matching struct padding
8f77bcd0
Fix correctness, keep perf - inspired by Base.padding.
3c1fefbb
Throw the right kind of error
99981941
Switch to depth-first search which is faster than recursion at compil…
7b86b5b6
Very minor compiler perf optimization: reuse the same work instead of
9dbd35bb
Simplify code: The new impl generalizes all the others.
9015f870
remove benchmark only code;
70035110
comments
063c737a
Start replacing Base's _reinterpret with our implementation.
dbf0507d
Don't mix Base.padding w/ new computations. This saves some compilati…
e5dc76ad
Small perf improvement to ispacked
f7964eef
Merge branch 'master' into nhd/reinterpret-padded-struct-performance
f3dd0b49
Comments: _packedsize, remove lint comment
c56e139d
Use the existing ArgumentErrors from master.
5b246605
Remove unnecessary macro
60942b4e
Add more tests for reinterpret
5cb44c3b
fixup merge conflict
53e69766
whitespace
34bd84ee
Add broken tests
16b20a81
Fix broken tests for nested structs
1da2607c
Perf improvement on the _packed_regions
6a0bd80f
Simplify the comment on the byte-cast optimization
28caa396
remove todo comment
b5e2e733
Make :foldable assertions tighter.
a3c7869f
cleanup from copilot
c5b9c19d
improve assert message
0ea5b495
Add broken test for missed bug;
64a93ecf
Fix bug for types that match prefix, but differ in trailing padding
92ad4a55
NHDaly
commented
on 2025-12-23
Update base/reinterpret.jl
7af9dab9
Fix reinterpret tests for 32-bit.
e303c350
NHDaly
commented
on 2026-01-03
Update base/reinterpret.jl
f213d55f
Assignees
No one assigned
Labels
performance
Milestone
No milestone
Login to write a write a comment.
Login via GitHub