Skip to content

[3.12] gh-113993: Make interned strings mortal (GH-120520, GH-121364, GH-121903, GH-122303) #123065

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

Merged
Yhg1s merged 14 commits into from
Sep 27, 2024
Merged

[3.12] gh-113993: Make interned strings mortal (GH-120520, GH-121364, GH-121903, GH-122303) #123065

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
9fd6334
[3.12] gh-113993: Allow interned strings to be mortal, and fix relate…
encukou Jun 24, 2024
9d4ade1
Put _Py_SetImmortal assertion in the old place
encukou Aug 15, 2024
1b8c12e
Create per-interpreter intern dict after init of deep-frozen modules
encukou Aug 15, 2024
aaa8afe
[3.12] gh-113993: Don't immortalize in PyUnicode_InternInPlace; keep …
encukou Jul 17, 2024
18b9ade
[3.12] gh-121863: Immortalize names in code objects to avoid crash (G…
encukou Jul 17, 2024
1d4a531
[3.12] gh-122291: Intern latin-1 one-byte strings at startup (GH-122303)
encukou Jul 27, 2024
cf6a27d
Adjust internal docs for 3.12
encukou Aug 15, 2024
fcee663
Regenerate the ABI dump (static strings are reorganized)
encukou Aug 19, 2024
0b10ee5
Don't link to the missing glossary term `immortal`.
encukou Aug 19, 2024
c405c0a
Merge branch '3.12' into mortal-interns-3.12
encukou Aug 19, 2024
2640dc8
Regenerate test_frozenmain.h
encukou Aug 20, 2024
50e1492
generate_global_objects.py: Fix name of macro in error message (#124464)
JelleZijlstra Sep 27, 2024
8583146
Merge in the main branch
encukou Sep 27, 2024
2d91ff9
Merge branch '3.12' into mortal-interns-3.12
Yhg1s Sep 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Adjust internal docs for 3.12
  • Loading branch information
@encukou
encukou committed Aug 16, 2024
commit cf6a27d0fe4302bae423d9d4e2ffc34dcf48608d
2 changes: 1 addition & 1 deletion Include/internal/pycore_unicodeobject.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ PyAPI_FUNC(void) _PyUnicode_InternMortal(PyInterpreterState *interp, PyObject **
PyAPI_FUNC(void) _PyUnicode_InternImmortal(PyInterpreterState *interp, PyObject **);
// Left here to help backporting:
PyAPI_FUNC(void) _PyUnicode_InternInPlace(PyInterpreterState *interp, PyObject **p);
// Only for singletons in the _PyRuntime struct:
// Only for statically allocated strings:
extern void _PyUnicode_InternStatic(PyInterpreterState *interp, PyObject **);

/* other API */
Expand Down
6 changes: 5 additions & 1 deletion InternalDocs/string_interning.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,11 @@ for declaration, initialization and finalization.

The empty string is one of the singletons: `_Py_STR(empty)`.

The three sets of singletons (`_Py_LATIN1_CHR`, `_Py_ID`, `_Py_STR`)
Deep-frozen modules (see `Tools/build/deepfreeze.py`) use either singletons,
or statically allocated strings. These are added to `INTERNED_STRINGS`
at runtime initialization, when deepfreeze modules are loaded.

These sets of singletons (`_Py_LATIN1_CHR`, `_Py_ID`, `_Py_STR`, deepfreeze)
are disjoint.
If you have such a singleton, it (and no other copy) will be interned.

Expand Down