Skip to content

Support for large numbers of args/kws with Python 3.10 #7866

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
sklam merged 24 commits into from
May 6, 2022
Merged

Support for large numbers of args/kws with Python 3.10 #7866

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7ad9642
added basic support for args <= 30 and kws <= 15
Feb 22, 2022
ebea710
Support for kws > 15
Feb 22, 2022
3107840
working with args > 30, needs refactoring + error checking
Feb 23, 2022
fd6669b
Refactored the code into helpful functions, still need to write tests
Feb 25, 2022
f3ad535
added tests
Feb 25, 2022
29953e7
added Ehsan's comments
Feb 28, 2022
28a22ee
fixed small bug in skip_unless_py10
Mar 1, 2022
f726b3a
updated the comments and tests
Mar 2, 2022
974da74
added an objmode test
Apr 18, 2022
dbb4dec
fixed the definition updates
Apr 18, 2022
7231d71
Add Siu's changes
Apr 18, 2022
2a01061
prepared to add extra tests
Apr 18, 2022
1d38efa
updated the error message and added additional tests
Apr 18, 2022
f766532
Added tests to check for inlined control flow, doesn't handle only args
Apr 18, 2022
7e8d424
added error handling for the inline case with all args
Apr 18, 2022
cd04a5e
simplified the error message sources
Apr 18, 2022
224c514
fix pytest assertion
Apr 18, 2022
9d234a2
updated the error message
Apr 28, 2022
af48701
Update numba/core/interpreter.py
njriasan Apr 28, 2022
9ed27ea
made Stuart's test changes
May 4, 2022
530284e
apply stuart's interpreter changes
May 4, 2022
9bb2ee5
Merge branch 'nick/16_kwargs_py310_peephole' of https://github.com/nj…
May 4, 2022
ba83ffb
applied remaining comment changes
May 4, 2022
265a426
Made Stuart's final changes
May 6, 2022
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
Update numba/core/interpreter.py 
Added sklam's typo fix

Co-authored-by: Siu Kwan Lam <1929845+sklam@users.noreply.github.com>
  • Loading branch information
@njriasan @sklam
njriasan and sklam authored Apr 28, 2022
commit af48701c35bf4f21ae27b3e4994e51f23ff1484f
2 changes: 1 addition & 1 deletion numba/core/interpreter.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ def _remove_assignment_definition(old_body, idx, func_ir):
func_ir._definitions[lhs].clear()
else:
raise UnsupportedError(
"Inconsistency found in the definitions while electuting"
"Inconsistency found in the definitions while executing"
" peep_hole_call_function_ex_to_call_function_kw. This suggests"
" an internal error or inconsistency elsewhere in the compiler."
)
Expand Down