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
Skip test for WASI
  • Loading branch information
@markshannon
markshannon committed May 3, 2024
commit 1831eb3939164dd9493f82668f86dffeb5b15135
7 changes: 1 addition & 6 deletions Include/cpython/pystate.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -193,12 +193,7 @@ struct _ts {
#ifdef Py_DEBUG
// A debug build is likely built with low optimization level which implies
// higher stack memory usage than a release build: use a lower limit.
# if defined(__wasi__)
// Based on wasmtime 16.
# define Py_C_RECURSION_LIMIT 300
# else
# define Py_C_RECURSION_LIMIT 500
# endif
# define Py_C_RECURSION_LIMIT 500
#elif defined(__s390x__)
# define Py_C_RECURSION_LIMIT 800
#elif defined(_WIN32) && defined(_M_ARM64)
Expand Down
5 changes: 3 additions & 2 deletions Lib/test/test_dynamic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
import sys
import unittest

from test.support import swap_item, swap_attr
from test.support import swap_item, swap_attr, is_wasi


class RebindBuiltinsTests(unittest.TestCase):
Expand Down Expand Up @@ -134,14 +134,15 @@ def test_eval_gives_lambda_custom_globals(self):

self.assertEqual(foo(), 7)

@unittest.skipIf(is_wasi and Py_DEBUG, "requires too much stack")
def test_load_global_specialization_failure_keeps_oparg(self):
# https://github.com/python/cpython/issues/91625
class MyGlobals(dict):
def __missing__(self, key):
return int(key.removeprefix("_number_"))

# Need more than 256 variables to use EXTENDED_ARGS
variables = 270
variables = 400
code = "lambda: " + "+".join(f"_number_{i}" for i in range(variables))
sum_func = eval(code, MyGlobals())
expected = sum(range(variables))
Expand Down