[Python-checkins] GH-93911: Fix `LOAD_ATTR_PROPERTY` caches (GH-96519)
markshannon
webhook-mailer at python.org
Tue Sep 6 07:12:10 EDT 2022
https://github.com/python/cpython/commit/cd0ff9bd14d6a60e841d10f8415827db556ae622
commit: cd0ff9bd14d6a60e841d10f8415827db556ae622
branch: main
author: Brandt Bucher <brandtbucher at microsoft.com>
committer: markshannon <mark at hotpy.org>
date: 2022-09-06T12:11:38+01:00
summary:
GH-93911: Fix `LOAD_ATTR_PROPERTY` caches (GH-96519)
files:
A Misc/NEWS.d/next/Core and Builtins/2022-09-02-16-47-52.gh-issue-93911.vF-GWe.rst
M Python/specialize.c
diff --git a/Misc/NEWS.d/next/Core and Builtins/2022-09-02-16-47-52.gh-issue-93911.vF-GWe.rst b/Misc/NEWS.d/next/Core and Builtins/2022-09-02-16-47-52.gh-issue-93911.vF-GWe.rst
new file mode 100644
index 000000000000..b8dc0435377b
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2022-09-02-16-47-52.gh-issue-93911.vF-GWe.rst
@@ -0,0 +1,2 @@
+Fix an issue that could prevent :opcode:`LOAD_ATTR` from specializing
+properly when accessing properties.
diff --git a/Python/specialize.c b/Python/specialize.c
index e8c3f468feaa..299adf34528a 100644
--- a/Python/specialize.c
+++ b/Python/specialize.c
@@ -775,8 +775,10 @@ _Py_Specialize_LoadAttr(PyObject *owner, _Py_CODEUNIT *instr, PyObject *name)
SPECIALIZATION_FAIL(LOAD_ATTR, SPEC_FAIL_ATTR_PROPERTY_NOT_PY_FUNCTION);
goto fail;
}
- uint32_t version = function_check_args(fget, 1, LOAD_ATTR) &&
- function_get_version(fget, LOAD_ATTR);
+ if (!function_check_args(fget, 1, LOAD_ATTR)) {
+ goto fail;
+ }
+ uint32_t version = function_get_version(fget, LOAD_ATTR);
if (version == 0) {
goto fail;
}
@@ -831,9 +833,7 @@ _Py_Specialize_LoadAttr(PyObject *owner, _Py_CODEUNIT *instr, PyObject *name)
assert(type->tp_getattro == _Py_slot_tp_getattro);
assert(Py_IS_TYPE(descr, &PyFunction_Type));
_PyLoadMethodCache *lm_cache = (_PyLoadMethodCache *)(instr + 1);
- uint32_t func_version = function_check_args(descr, 2, LOAD_ATTR) &&
- function_get_version(descr, LOAD_ATTR);
- if (func_version == 0) {
+ if (!function_check_args(descr, 2, LOAD_ATTR)) {
goto fail;
}
/* borrowed */
More information about the Python-checkins
mailing list