Thanks to visit codestin.com
Credit goes to github.com

Skip to content

GH-118095: Use broader specializations in tier 1, for better tier 2 support of calls. #118322

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
markshannon merged 19 commits into from
May 4, 2024
Merged

GH-118095: Use broader specializations in tier 1, for better tier 2 support of calls. #118322

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
0b157ab
Add CALL_PY_GENERAL and CALL_BOUND_METHOD_GENERAL call specializations.
markshannon Apr 26, 2024
2262f98
Add CALL_NON_PY_GENERAL call specialization
markshannon Apr 26, 2024
4af43c2
Remove CALL_PY_WITH_DEFAULTS specialization
markshannon Apr 26, 2024
b7795e3
Fix JIT build
markshannon Apr 26, 2024
e5c7dce
Use CALL_NON_PY_GENERAL in more cases when otherwise failing to speci…
markshannon Apr 30, 2024
61abb3b
Delete unused code
markshannon May 2, 2024
b49bc86
Make sure that function version number is checked when tracing throug…
markshannon May 3, 2024
f42519d
whitespace
markshannon May 3, 2024
3294632
Merge branch 'main' into tier-2-call
markshannon May 3, 2024
1a2dcfe
Check verion on function, not method
markshannon May 3, 2024
62ff43c
Lower WASI C recursion limit again
markshannon May 3, 2024
71c6d41
Get test passing on wasi
markshannon May 3, 2024
0b40773
Lower WASI C recursion limit again
markshannon May 3, 2024
02cdf2f
Yet another attempt to get the WASI build to work
markshannon May 3, 2024
1831eb3
Skip test for WASI
markshannon May 3, 2024
7819b1c
Skip another test for WASI
markshannon May 3, 2024
857d152
Address some review comments
markshannon May 4, 2024
839d16e
Use _SAVE_RETURN_OFFSET in general call instructions
markshannon May 4, 2024
1d5c0a7
Merge branch 'main' into tier-2-call
markshannon May 4, 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
Use _SAVE_RETURN_OFFSET in general call instructions
  • Loading branch information
@markshannon
markshannon committed May 4, 2024
commit 839d16e009c57c379b5dfc1fa1387a6a2259b49e
4 changes: 2 additions & 2 deletions Include/internal/pycore_opcode_metadata.h
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

222 changes: 111 additions & 111 deletions Include/internal/pycore_uop_ids.h
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions Include/internal/pycore_uop_metadata.h
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 6 additions & 5 deletions Python/bytecodes.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3148,7 +3148,7 @@ dummy_func(

macro(CALL) = _SPECIALIZE_CALL + unused/2 + _CALL + _CHECK_PERIODIC;

op(_CALL_PY_GENERAL, (callable, self_or_null, args[oparg] -- new_frame: _PyInterpreterFrame*)) {
op(_PY_FRAME_GENERAL, (callable, self_or_null, args[oparg] -- new_frame: _PyInterpreterFrame*)) {
// oparg counts all of the args, but *not* self:
int total_args = oparg;
if (self_or_null != NULL) {
Expand All @@ -3168,7 +3168,6 @@ dummy_func(
if (new_frame == NULL) {
ERROR_NO_POP();
}
frame->return_offset = (uint16_t)(1 + INLINE_CACHE_ENTRIES_CALL);
}

op(_CHECK_FUNCTION_VERSION, (func_version/2, callable, unused, unused[oparg] -- callable, unused, unused[oparg])) {
Expand All @@ -3181,7 +3180,8 @@ dummy_func(
unused/1 + // Skip over the counter
_CHECK_PEP_523 +
_CHECK_FUNCTION_VERSION +
_CALL_PY_GENERAL +
_PY_FRAME_GENERAL +
_SAVE_RETURN_OFFSET +
_PUSH_FRAME;

op(_CHECK_METHOD_VERSION, (func_version/2, callable, null, unused[oparg] -- callable, null, unused[oparg])) {
Expand All @@ -3197,7 +3197,7 @@ dummy_func(
assert(Py_TYPE(callable) == &PyMethod_Type);
self = ((PyMethodObject *)callable)->im_self;
Py_INCREF(self);
stack_pointer[-1 - oparg] = self; // Patch stack as it is used by _CALL_PY_GENERAL
stack_pointer[-1 - oparg] = self; // Patch stack as it is used by _PY_FRAME_GENERAL
method = ((PyMethodObject *)callable)->im_func;
assert(PyFunction_Check(method));
Py_INCREF(method);
Expand All @@ -3209,7 +3209,8 @@ dummy_func(
_CHECK_PEP_523 +
_CHECK_METHOD_VERSION +
_EXPAND_METHOD +
_CALL_PY_GENERAL +
_PY_FRAME_GENERAL +
_SAVE_RETURN_OFFSET +
_PUSH_FRAME;

op(_CHECK_IS_NOT_PY_CALLABLE, (callable, unused, unused[oparg] -- callable, unused, unused[oparg])) {
Expand Down
Loading