Skip to content

Introduce zend_ast_call_get_args() - #20859

Merged
arnaud-lb merged 3 commits into
php:masterfrom
arnaud-lb:ast-call-get-args
Jan 8, 2026
Merged

Introduce zend_ast_call_get_args()#20859
arnaud-lb merged 3 commits into
php:masterfrom
arnaud-lb:ast-call-get-args

Conversation

@arnaud-lb

@arnaud-lbarnaud-lb commented Jan 7, 2026

Copy link
Copy Markdown
Member

See #20848 (comment).

Introduce zend_ast_call_get_args(). The function is useful to fetch the arg node of any call node.

Also refactor zend_ast_evaluate_inner() to use it.

The function will be used in #20848, in zend_compile_pipe().

@arnaud-lbarnaud-lb mentioned this pull request Jan 7, 2026
Merged
@arnaud-lb
arnaud-lb marked this pull request as ready for review January 7, 2026 15:28
iluuu1994
iluuu1994 approved these changes Jan 7, 2026
View reviewed changes

@iluuu1994iluuu1994 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✔️ otherwise

Comment threadZend/zend_ast.c
@arnaud-lb
arnaud-lb merged commit ef52252 into php:masterJan 8, 2026
10 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TimWollaTimWollaTimWolla approved these changes
@iluuu1994iluuu1994iluuu1994 approved these changes

Assignees

No one assigned

Labels

ABI breakCategory: Engine

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@arnaud-lb@TimWolla@iluuu1994