Cleanup SPL instantiation code

Fix #78236: convert error on receiving variables when duplicate [

When an input variable name contains a non matched open bracket, we not
only have to replace that with an underscore, but also all following
forbidden characters.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79884: PHP_CONFIG_FILE_PATH is meaningless

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79884: PHP_CONFIG_FILE_PATH is meaningless

Fix #79884: PHP_CONFIG_FILE_PATH is meaningless

It does not make sense to make assumptions about `PHP_CONFIG_FILE_PATH`
during build time, since that value is never used during run time on
Windows.  Since there is no `--with-config-file-path` on Windows
either, we define `PHP_CONFIG_FILE_PATH` as `""`.

Don't record "fake" closures

Remove no longer needed stdint compatibility defines

Closes GH-5886.

We track only arguments of user functions on abstract stack

Fixed 32-bit build

Revert "Fixed DynASM failure"

This reverts commit 62ca6b567cdae0f8f7e02e6e2ebe6bb64ee1dd3c.

Fixed DynASM failure

Avoid register reloading

Fix some clang warnings

Try to fix 32-bit NTS build

Make test decimal to binary conversion expectation more tolerant

Fix cut/paste error in expected test output

Fix build

MSVC is pretty picky there, so we add the `ZEND_VOIDP` cast.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix brittle test

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix brittle test

Fix brittle test

This test fails occasionally due to timing issues, because the session
file may have been unlinked by the first `session_start()`'s GC.  We
adapt the test expectation to this reality.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix minor memory leak of CG(script_encoding_list)

Fix minor memory leak of CG(script_encoding_list)

This only leaks at the end of the process, so per se not an issue, but
the leak is caught by MSVC's CRT leak checker, so we better properly
clean up to avoid false positives.

Revert "Disable tracing JIT on AppVeyor"

This reverts commit a986933ddeff664367d11bebaca2ea6977dbce62.

Tracing JIT for Windows is already fixed.

Fix warnings of strict-prototypes

Closes GH-5887.

Change type of max_num_args to uint32_t

Closes GH-5885.

Export zend_register_error_notify_callback() with ZEND_API

Accept unsigned instead of enum pdo_attribute_type

Driver-specific options get passed to these functions, which are
not part of the pdo_attribute_type enum. Newer GCC versions don't
like that. Accept a simple integr type instead.

Reserve WIN64 shadow space

php.ini cleanup

[ci skip]

Separate and fix deoptimization code generator

Treat namespaced names as single token

Namespace names are now lexed as single tokens of type
T_NAME_QUALIFIED, T_NAME_FULLY_QUALIFIED or T_NAME_RELATIVE.

RFC: https://wiki.php.net/rfc/namespaced_names_as_token

Closes GH-5827.

Improved number to string comparison semantics

RFC: https://wiki.php.net/rfc/string_to_number_comparison

Closes GH-3886.

Prevent incorrect trasing JIT TSSA usage for range() result type inference

Check dual_it validity in CallbackFilterIterator::accept()

Avoid accessing intern->u.cbfilter null pointer, though it's
harmless here.

Drop no-op dasm_growpc() call

This doesn't do anything, as maxpc=0 is passed. However, ubsan
complains about the memset of a null pointer, so avoid it.

Mark zend_jit_patch memory accesses as unaligned

This prevents ubsan from complaining.

Move checks after zpp in opcache_is_script_cached()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79881

Fixed bug #79881

Update NEWS

Use ZPP callable check for oci_register_taf_callback()

Some cleanup in OCI8 extension for PHP 8

Disable tracing JIT on AppVeyor

This is a temporary measure until the current issues[1] are resolved.

[1] <https://github.com/php/php-src/commit/0e1e991a89528b571a0271a3f24f9d3c2c36e23a#commitcomment-40805659>

Fix deoptimization info and corresponding memory leaks

Fixed bug #79864 (JIT segfault in Symfony OptionsResolver)

Support optional comma in closure use list

RFC: https://wiki.php.net/rfc/trailing_comma_in_closure_use_list
Discussion: https://externals.io/message/110715

The release manager has agreed to allow merging of RFCs that have near-unanimous
votes. If an RFC ends up not achieving the required 2/3 majority at the time the
announced voting period closes, this implementation commit will be reverted
in time for the feature freeze.

Closes GH-5793

Improve argument types and names in ext/ctype

Closes GH-5878

Give zend_pass_function an arginfo

Now that the ZEND_ACC_VARIADIC flag is set, we should also make
sure there is a variadic arg that can be looked up.

Tweak zend_test arginfo/zpp

Fix incorrect initialization in date_sunrise()

Switch to tracing JIT by default

Fix arg/func info

 * mysqli_get_server_info() cannot return null. The underlying API
   is infallible.
 * mysqli_select_db() func info is redundant.
 * mb_detect_order() can only return array|true, not array|false.

Also make the func_info.phpt test that is supposed to catch these
kinds of issues actually work.

Cleanup argument handling of Zend functions and methods

Closes GH-5847

add ZipArchive::CM_XZ constant (in upcoming libzip 1.8.0)

Fix type recording and side exit information for FE_FETCH_* instructions

Run arginfo / zpp mismatch tests for methods as well

As a side-effect, this also catches issues where classes are not
prepared for use with newInstanceWithoutConstructor.

Check for missing arginfo arguments

Internal functions error when too many arguments are passed. Make
this part of the verification we do in debug builds. This will
help avoid cases where an argument is missing in the stubs,
as recently encountered in 6d96f0f.

Avoid recording of uninitialized variable

Check for redundant func info

Separate __call and __soapCall implementations

This is overly pedantic, but allows us to enable more arginfo
consistency checks.

Add stubs for PDO PGSql extension methods

Add stubs for PDO SQLite extension methods

Putting these under a dummy PDO_SQLite_Ext class.

Disable JIT for PHPDBG

Fix UConvert::getErrorMessage() leak on zpp failure

Move SOAP_SERVER_BEGIN_CODE() after zpp

This is still very dubious, because there are lots of other "returns"
between the BEGIN and END -- won't that end up not restoring the
original state?

Make ReflectionGenerator final

This class is not safe against malicious extension / instantiation.

Stricter verification of func info against arg info

Make sure they're actually the same up to cases where func info
allows more accurate expressions. There are some awkward edge cases
around true/false/null limitations in union types.

Adjust Doctrine workaround in community job

[ci skip]

Switch to tracing JIT by default

Use different temporary register

Next is PHP 8.0.0beta1

Remove some redundant func info

This is fully covered by arginfo.

The array_merge_recursive() RC information was also wrong,
it should be the same as array_merge().

Update NEWS for PHP 8.0.0alpha3

Fixed support for 64-bit constants

Fix mb_ereg_search* arg and func info

Fix readline test after callback changes

This one requires libedit and was probably missed for that reason.

Remove null from highlight_* return types

Also fix show_source() discrepancy in func_info.

Merge branch 'PHP-7.4'

Prepare for 7.4.10

Remove bool return type from assert_options

Not seeing any way this function can return bool.

Fix iptcembed func info

This function can return true with $spool >= 2.

Call zpp_none in PharFileInfo::__destruct()

Using __destruct() with internal classes is dubious, but not so
simple to avoid here because the code extends SPL classes.

Merge branch 'PHP-7.4'

* PHP-7.4:
  7.3 is now 7.3.22-dev

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  7.3 is now 7.3.22-dev

7.3 is now 7.3.22-dev

Fixed pg_select() function info

Use ZPP callable check in readline extension

Use ZPP callable for PDO Sqlite

Fixed tracing JIT exception handling

Explode _php_image_output_ctx() into multiple functions

Closes GH-5844

Fixed VM interrupt handling

Keep CPU regesters used by deoptimizer

Fixed tracing JIT for VM without global register variables

Preserve original ce_flags when registering class

Bug that regularly sneaks in: ZEND_ACC_FINAL is set before calling
zend_register_internal_class() and promptly gets ignored. Remove
this footgun by preserving flags from the original CE.

Fetch pdo stmt after zpp

Update method signature in test

Use zpp for PDO fetch mode

Also changing the function signatures to accept variadic args
for the fetch params. If we're already breaking Doctrine anyway,
we may as well do it properly.

Fix bug #79857: Add upgrading note for exit dtor change

[ci skip]

Fix bug #79868

This simply restores the code from PHP 7.4 which I incorrectly
"simplified" in master.

Add common code for magic method assignment

This was repeated three times.