Skip to content
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.

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

Reference symbols from global space via FQN #7

Closed
wants to merge 2 commits into from

Conversation

lippserd
Copy link
Member

PHP resolves relative function and constant references at call time. For each function call or access to a constant PHP checks whether the function or constant belongs to the current namespace or the global space. PHP7 introduces opcode optimizations which only apply for fully qualified name (FQN) calls.

@lippserd lippserd force-pushed the feature/reference-global-symbols-via-fqn branch 2 times, most recently from d7467da to 9b6989f Compare January 18, 2019 08:19
Motivation: PHP resolves relative function references at call time. For
each function call PHP checks whether the function belongs to the
current namespace or the global space. PHP7 introduces opcode
optimizations which only apply for fully qualified name (FQN) calls.
Motivation: Since we use FQN calls for functions from the global space
it makes sense to do that for classes too.
@lippserd lippserd force-pushed the feature/reference-global-symbols-via-fqn branch from 9b6989f to 160c3b0 Compare January 18, 2019 15:34
@lippserd
Copy link
Member Author

@lippserd lippserd added the wontfix This will not be worked on label Oct 1, 2019
@lippserd
Copy link
Member Author

lippserd commented Oct 1, 2019

Performance improve is insignificant. I'll close this one.

@lippserd lippserd closed this Oct 1, 2019
@lippserd lippserd deleted the feature/reference-global-symbols-via-fqn branch February 14, 2020 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant