Implement "Constructor Promotion" RFC

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

Closes GH-5291.

micro-optimization

Drop hard-coded /W3 default in favor of custom CFLAGS

Building with `/W3` shows an awful lot of warnings on Windows, so it's
really hard to spot the more important ones.  Since it is not possible
to override the hard-coded `/W3`, we drop it altogether, so MSVC uses
the default `/W1`.  Users are encouraged to increase the warning level
via the environment variable `CFLAGS` before doing configure.

We also enable `/WX` (treat warnings as errors) for AppVeyor CI, using
`/W1` for now, since otherwise the build would fail.

Fix MSVC level 1 (severe) warnings

We fix (hopefully) all instances of:

* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4005>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4024>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4028>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4047>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4087>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4090>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4273>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4312>

`zend_llist_add_element()` and `zend_llist_prepend_element()` now
explicitly expect a *const* pointer.

We use the macro `ZEND_VOIDP()` instead of a `(void*)` cast to suppress
C4090; this should prevent accidential removal of the cast by
clarifying the intention, and makes it easier to remove the casts if
the issue[1] will be resolved sometime.

[1] <https://developercommunity.visualstudio.com/content/problem/390711/c-compiler-incorrect-propagation-of-const-qualifie.html>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update intl test suite for ICU 67.1

Update intl test suite for ICU 67.1

Cherry-picked from c915c601710e2a44e6c7f89fcb12b8047c968108.

Free attribute validators on shutdown

Don't leak attributes on internal classes

Also add zend_hash_release() API to complement zend_array_release(),
because the latter is specific to non-persistent zval arrays.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Respect --program-prefix/suffix when installing phar

Respect --program-prefix/suffix when installing phar

Currently ./configure --enable-phar --program-suffix=7.4 will
result in binaries named php7.4 and phar but should instead
result in php7.4 and phar7.4

Closes GH-5650.

Pass zend_string message to zend_error_cb

This makes the zend_error_cb API simpler, and avoid formatting
the same message in multiple places.

It should be noted that the passed zend_string is always
non-persistent, so if you want to store it persistently somewhere,
you may still need to duplicate it.

The last_error_message is cleared a bit more aggressive, to make
sure it doesn't hang around across allocator life-cycles.

Closes GH-5639.

Fixed bug #72089: Throw Error on require failure

It should be noted that we still throw the usual fopen warnings,
but the final fatal error becomes an Error exception. Combine
with @ to suppress FS warnings.

Closes GH-5641.

Fix phpdbg watchpoints wrt. negative_array_index RFC

The implementation of that RFC changed the initial value of
`zend_array.nNextFreeElement` to `-1`; we work around that by inserting
first, and retrieving the index afterwards.

We also fix the erroneous printf specifier for the unsigned integer.

Avoid useless "mov"

Fix weird zend_bool usage in Intl Calendar::roll() method

This code really needs to be review as it's convoluted for no good reason.

in_array() avoid internal property access as we have the arrlen already

Closes GH-5662

Add upgrading note for Attributes RFC

[ci-skip]

Add Attributes

Co-authored-by: Martin Schröder <m.schroeder2007@gmail.com>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #73527: Invalid memory access in php_filter_strip

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #73527: Invalid memory access in php_filter_strip

Fix #73527: Invalid memory access in php_filter_strip

Add guard for FETCH_DIM_FUNC_ARG

Prefer shorter x86 instructions

Implement #47074: phpinfo() reports "On" as 1 for the some extensions

What is modified as boolean, should also be displayed as boolean.

Improved tracing JIT for FETCH_OBJ_R/IS

Remove MYSQLND_COMPRESSION_WANTED define

Instead directly define MYSQLND_COMPRESSION_ENABLED. This also
matches what config.w32 does.

Merge branch 'PHP-7.4'

* PHP-7.4:
  mysqlnd: Remove unnecessary check for HAVE_ZLIB

mysqlnd: Remove unnecessary check for HAVE_ZLIB

If MYSQLND_COMPRESSION_WANTED is set, we already guarantee that
zlib is linked, but don't necessarily set HAVE_ZLIB.

Closes GH-5658.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79668: get_defined_functions(true) may miss functions

Tracing JIT for FETCH_DIM_FUNC_ARG and FETCH_OBJ_FUNC_ARG when they are used in READ mode (to pass by value).

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79668: get_defined_functions(true) may miss functions

Fix #79668: get_defined_functions(true) may miss functions

Instead of some brittle and unefficient string matching, we can just
check for the function handler.

Fixed abstract stack consistency for JMPZ_EX/JMPNZ_EX

"call_info" doesn't matter

Avoid useless register reload

Fix typo in skipif section

Merge mistake...

Avoid useless REFCOUNTED check

Fix #79665: ini_get() and opcache_get_configuration() inconsistency

Overriding the given INI values in modifier callbacks is not possible,
so instead of enforcing "normalized" internal values, we just reject
the attempted changes.

Use cheaper zend_rethrow_exception() instead of zend_throw_exception_internal()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Report len as -1 instead of INT_MAX

Report len as -1 instead of INT_MAX

Per docs it should be -1. And would be on 32-bit systems, but
not on 64-bit systems.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Skip new watch point test under asan

Skip new watch point test under asan

Handle VM interrupts after DO_ICALL through side exits

%d out token IDs in test

Avoid having to update this test any time token numbers shift.

Avoid reference counting when RETURN CV

Added missed helper

Skip new test case on Windows when JIT is enabled

Cf. <http://git.php.net/?p=php-src.git;a=commit;h=c5cf0af8a98cbc574fd315bf9d78033b896886f3>.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #73927: phpdbg fails with windows error prompt at "watch array"

Fix #73927: phpdbg fails with windows error prompt at "watch array"

We expect zvals, so we should request zvals.

We also suppress spurious watchpoint removal notices.

Tracing JIT support for indirect CV modification (may be incomplete)

Afdded misse MAY_BE_ARRAY_* flags

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79664: PDOStatement::getColumnMeta fails on empty result set

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79664: PDOStatement::getColumnMeta fails on empty result set

Fix #79664: PDOStatement::getColumnMeta fails on empty result set

As its name suggests, `sqlite3_data_count` returns the number of
columns in the current row of the result set; we are interested in the
number of columns regardless of the current row, so we have to use
`sqlite3_column_count` instead.

Add some mixed types

Fixed retutn type inference when return undefined variable

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79650: php-win.exe 100% cpu lockup

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79650: php-win.exe 100% cpu lockup

Fix #79650: php-win.exe 100% cpu lockup

As of PHP 7.3.0, `sapi_cli_single_write()` is supposed to return `< 0`
on failure, but `fwrite()` returns a `size_t`, and signals error by
setting the stream's error indicator.  We have to cater to that.

Fixed tracing JIT for closure run_time_cache access

Fix php_get_args function info return type and add arg check

Closes GH-5648.

Add UPGRADING note for pty support

[ci skip]

Add UPGRADING note for xmlrpc unbundling

And drop other UPGRADING references to xmlrpc. I don't think it
makes sense to have those for extensions that are no longer
shipped...

[ci skip]

Fix some line overruns in UPGRADING

[ci skip]

Move upgrading note for mixed

This should be in the "new features" section.

[ci skip]

Fix small typo in comment

update node-fetch url

Fix #79653 Unexpected error for ArrayIterator::ksort()

Closes GH-5644

Use ZPP callable check for spl_autoload_register.

This makes it always throw a TypeError, moreover this makes the
error message consistent.

Added a warning mentioning that the second parameter is now ignored
when passed false.

Closes GH-5301

Fix #79652 Ensure that the mixed type is displayed instead of the union of all types

Closes GH-56430

Unbundle ext/xmlrpc

According to <https://wiki.php.net/rfc/unbundle_xmlprc> we unbundle
ext/xmlrpc.

Add upgrading notes about the deprecated functionality in ext/enchant

[skip ci]

Convert enchant resources to opaque objects

Additionally, deprecate ENCHANT_MYSPELL and ENCHANT_ISPELL constants.
Closes GH-5577

Co-authored-by: Remi Collet <remi@php.net>

Support libzip 1.6 on Windows

Fix Windows build

Fixed tracing JIT register allocator (failure on ext/opcache/tests/jit/assign_002.phpt)

Allow tracing JIT generate code when function exits from VM (e.g. for magic __get/__set)

Remove json checks in fuzzer SAPI

JSON is now always enabled

Don't add guards that are not going to be checked

Fixed use-after-free in tracing JIT when recording closures and top-level op_arrays.

Clean up calls to extension_loaded('json') in tests

These are no longer needed after https://wiki.php.net/rfc/always_enable_json

Closes GH-5637

Fixed tracing JIT type inference for passing undefined CV by value

Fixed incorrect trace type inference for top-level code

Fixed incorrect type inference

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix PVS-Studio integration

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix PVS-Studio integration

Fix PVS-Studio integration

Without this configuration option, PVS-Studio looks for preprocessed
files (*.i), but these do not exists.

Build fix for newer versions of NetBSD.

its libutil contains newer conflicting apis as estrdup, emalloc.

Closes GH-5635

Ensure Exception::getFile/getLine return type is correct

These return an untyped protected property, so we can't rely on
the type being correct.

Also add return types to the interface -- normally this would be
a no-go, but Throwable is a special interface that can only
be implemented internally, so we control all implementations.

Run Symfony tests in per-component

Re-enable Symfony test on PHP 8

Make Exception::$previous a typed property

Exception::$previous is a private property, so we can add a type:

    private ?Throwable $previous = null;

Make Exception::$trace typed array property

This is a private property, so we are allowed to add a type.
The new declaration of the property is:

    private array $trace = [];

This ensures that Exception::getTrace() does indeed return an array.

Userland code that was modifying the property through refleciton
may have to be adjusted to assign an array (instead of null,
for example).

Closes GH-5636.

Avoid recording of uninitialized types.
VM doesn't set Z_TYPE() for IS_VAR when passes class_entry reference.

Add some mixed parameter types and fix UNKNOWN default values in ext/xmlrpc

Extract code for declaring Exception/Error properties

Convert Exception::getMessage() result to string

We specify that the return type of Exception::getMessage() is a
string. However, we don't currently ensure this, because
Exception::$message is a protected member that can be set to any
type. Fix this by performing an explicit type-cast.

This also requires a temporary refcount increment in the __toString()
object handler, because there is no additional owner of the object,
and it may get released prematurely as part of the __toString() call.

Split "opcache.jit_max_recursion_unroll" into "opcache.jit_max_recursive_calls" and "opcache.jit_max_recursive_returns".
It's possible to disable recording of "recursive return loops" setting opcache.jit_max_recursive_returns to 0.

Fix func info for sodium_crypto_aead_xchacha20poly1305_ietf_decrypt