-
-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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.
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
GH-123516: Improve JIT memory consumption by invalidating cold executors #124443
Changes from all commits
0eac77b
d576296
68e95d6
c903af4
5ca6b7f
beb4f65
427dbf5
92d5590
0cdf638
58e7447
6c047e4
2645023
7c7ae98
6d6d306
4d086fe
d08e45a
6315877
622c266
e5117b2
7c6704c
1d72fdd
1778185
2506821
fca6dec
29436fd
b969b11
a669e0f
deb73ec
9fa55e8
e4a461a
d5a2bed
d232e63
e4a456a
b7d2d5a
6dcd2dc
219f890
fb7b04d
ea397a3
310d20c
d2f9dc4
d755d56
c9534c0
61cd3c5
e5065ad
2d09259
d2e8e29
a894598
758ee03
1927bfe
8ee0d7f
cedd65d
0a9b5b6
180a68e
7cb9cba
8939ecf
00f03d2
4981ab2
3c59316
306c3c3
fe50615
d51817b
131618a
5961103
99262b6
230fe05
f3c01a1
563a4d7
062c54f
17ece50
beea8c6
eb48f82
34363f2
77e81d4
09e3300
8ff071f
f238189
ec99d5a
9526d84
18febc7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
Improved JIT memory consumption by periodically freeing memory used by infrequently-executed code. | ||
This change is especially likely to improve the memory footprint of long-running programs. |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a thought I had while reading through... I don't think any of the stuff that manipulates the linked list of executors is thread-safe currently. Probably not a problem for this PR, but in the future we'll probably want to go through and add
_PyEval_StopTheWorld
and_PyEval_StartTheWorld
calls inoptimize.c
.