Implement array access notice in JIT

Merge branch 'PHP-7.4'

Throw notice on array access on illegal type

No notice is thrown for list() accesses, because we did not come
to an agreement regarding patterns like

    while ([$key, $value] = yield $it->next()) { ... }

where silent null access may be desirable.

No effort is made to suppress multiple notices in access chains
likes $x[0][0][0], because the technical complexity this causes
does not seem worthwhile.

RFC: https://wiki.php.net/rfc/notice-for-non-valid-array-container

Fix invalid array access in mysqli_fork test

Reduce cost for GC references to strings and resources in JIT

Merge branch 'PHP-7.4'

* PHP-7.4:
  Reduce cost for references to strings and resources

Reduce cost for references to strings and resources

Merge branch 'PHP-7.4'

Rename support_strings to !is_list

In preparation for more behavior that depends on whether it it is
a list() access. Using support_strings for that would be
confusing.

Simplify two unserialize() tests

unserialize() returns false for these now, so it doesn't make sense
to try and do something with the result.

Fix iteration limits in SXE test

Test doesn't look like it's specifically testing out of bounds
access, just two different iteration patterns, so make sure the
limits are correct.

Clean up DateTimeZone::getLocation() test

And avoid invalid array access.

Avoid invalid array access in fcgi.inc

Merge branch 'PHP-7.4'

* PHP-7.4:
  improve libargon2/libsodium compatibility

improve libargon2/libsodium compatibility

Merge branch 'PHP-7.4'

* PHP-7.4:
  Use e.g. instead of less common f.e. [ci skip]

Use e.g. instead of less common f.e. [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  Relax argon2 mem_cost down to 64k, bump time_cost to 4

Relax argon2 mem_cost down to 64k, bump time_cost to 4

Merge branch 'PHP-7.4'

CGI: Fix memory leak on error paths

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid unintended inlining

Avoid unintended inlining

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Revert "Fixed bug #76980"

This reverts commit 35353dc49a73a58c17c7896c4c4c3997ef2c007d.

This changes causes issues for Symfony, see
https://github.com/symfony/symfony/issues/32395. I'm reverting it
from PHP 7.2 and PHP 7.3 and only leaving it in PHP 7.4.

Merge remote-tracking branch 'origin/PHP-7.4' into PHP-7.4

Merge branch 'PHP-7.4'

* PHP-7.4:
  Upgrade to SQLite 3.28.0

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Upgrade to SQLite 3.28.0

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Upgrade to SQLite 3.28.0

Merge branch 'PHP-7.1' into PHP-7.2

* PHP-7.1:
  Upgrade to SQLite 3.28.0

Upgrade to SQLite 3.28.0

Over the years, multiple security vulnerabilities[1] have been found
and fixed in SQLite3, so it makes sense to update our bundled libsqlite
to the latest available version.

[1] <https://www.cvedetails.com/vulnerability-list/vendor_id-9237/Sqlite.html>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove duplicated code for (PRE|POST)_DEC_(OBJ|STATIC_PROP) opcodes

Remove duplicated code for (PRE|POST)_DEC_(OBJ|STATIC_PROP) opcodes

Update NEWS for 7.4.0beta1

Update NEWS for PHP 7.4.0alpha3

Merge branch 'PHP-7.4'

* PHP-7.4:
  Catch up with current procedures

Catch up with current procedures

- Git is used for moving existing extensions to PECL
- PHP versions with the PHP 7.4 examples
- Some minor tweaks here and there
- Mention re2c also
- Mention Docker image as a helper tool

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add zend.exception_ignore_args [ci skip]

Add zend.exception_ignore_args [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update NEWS [ci skip]
  Update NEWS [ci skip]
  Update NEWS [ci skip]

Update NEWS [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update NEWS [ci skip]
  Update NEWS [ci skip]

Update NEWS [ci skip]

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Update NEWS [ci skip]

Update NEWS [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed dispatch from inlined hybrid opcode to another opcode

Fixed dispatch from inlined hybrid opcode to another opcode

Merge branch 'PHP-7.4'

Add ReflectionReference::getRefcount()

And don't return null for rc=1 references. Leave it to the user
to decide whether or not they want to consider these as references
or not.

Fixes bug #78263.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Catch up with recent changes [ci skip]

Catch up with recent changes [ci skip]

- RFC about password_hash portability improvements added
- build system minor updates noted
- typos

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove APACHE symbol

Remove APACHE symbol

The APACHE symbol was used in very early PHP versions to indicate the
Apache module usage. Since PHP 4 this is no longer used in the code.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Refactor genif.sh

Refactor genif.sh

Changes:
- Coding style fixes
- ${1+"$@"} replaced with "$@" [1]
- EXTRA_MODULE_PTRS variable is not used anymore
- awk tool defined via environment variable
- srcdir determined automatically from the genif.sh location

[1] https://www.in-ulm.de/~mascheck/various/bourne_args/

Closes GH-4372

Merge branch 'PHP-7.4'

Make busy wait busier

Another stab in the dark to fix the intermittent failures of timeout
tests on macos CI: We're using ITIMER_PROF, which means that the
timer counts against user+system time. The "busy" wait loop counts
against real time. Currently it calls microtime() on every iteration.
If that call is implemented as a syscall rather than going through
vDSO or commpage we might be seeing many context switches here which
drive up the real time, but not user or system time.

See if making the loop busier and calling microtime() less helps the
situation.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Delay EG(exception) check on slow path

Delay EG(exception) check on slow path

Merge branch 'PHP-7.4'

Remove redundant variable declaration

Deprecate passing invalid character to base_convert etc

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

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove C89 checks for signal.h and strerror

Remove C89 checks for signal.h and strerror

These are part of the C89 and on today's systems not needed to be
checked anymore. This removes symbols HAVE_SIGNAL and HAVE_STRERROR.

- http://port70.net/~nsz/c/c89/c89-draft.html
- locale.h is also part of C89 but will be removed per request in PHP 8

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update libmagic patch

JIT: Reading php binary symbols list on FreeBSD

Closes GH-4363.

Update libmagic patch

This simplifies the libmagic patch:
- in upstream the HAVE_STRERROR check has been removed
- in upstream library the HAVE_SIGNAL_H has been removed
- indentations syncs with the upstream library
- some irrelevant changes removed from the patch (log comment), upstream
  has this correctly logged already so no need to patch the comment.

Closes GH-4298

Remove INPUT_SESSION and INPUT_REQUEST

These were never supported and always generated a warning. Remove
them entirely for PHP 8.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Implement FR #77230: Support custom CFLAGS and LDFLAGS from environment

Implement FR #77230: Support custom CFLAGS and LDFLAGS from environment

While it is already possible to *set* CFLAGS and LDFLAGS (actually all
variables) from the environment for `nmake` (by passing the `/E`
option), it is not possible to *add* any (C|LD)FLAGS, which can be
useful in some cases.  Instead of allowing this for `nmake`, we add
support for additional custom (C|LD)FLAGS to `configure`, similar to
how that works on Linux, so one could actually write:
````
set CFLAGS=foo & set LDFLAGS=bar & configure
````
This also allows us to use these flags during configure.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix TsHashTable related race conditions

Fix TsHashTable related race conditions

Although TsHashTable and the according API are supposed to easily make
a HashTable thread-safe, they do not; for instance, there can be race
conditions between finding and updating entries.  We therefore avoid
the usage of TsHashTable in favor of a HashTable with our own mutex
management.

The patch has been provided by krakjoe@php.net; I only did some minor
fixes and tweaks.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Simplify PHP_CHECK_PDO_INCLUDES calls

Simplify PHP_CHECK_PDO_INCLUDES calls

Conditional checks were once used for backwards compatibility with
phpize from PHP versions that didn't have this macro call yet.

Closes GH-4376

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed opcode description

Fixed opcode description

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove some unused variables

Remove some unused variables

- Variables php_abs_top_srcdir php_abs_top_builddir are no longer used.
- ZEND_EXT_TYPE is always zend_extension and variable is no longer used.

Closes GH-4378

Merge branch 'PHP-7.4'

* PHP-7.4:
  Provide argon2i(d) password hashing from sodium when needed

Provide argon2i(d) password hashing from sodium when needed

Merge branch 'PHP-7.4'

* PHP-7.4:
  Simplify expression and remove the possibility of div by 0

Simplify expression and remove the possibility of div by 0

Maybe should use exp2() but not sure about how supported it is.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove PHP_DEBUG_MACRO

Remove PHP_DEBUG_MACRO

The macro is no longer used. The warning at the end of the configure
script therefore is also no longer used.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove some more Apache 1 left overs

Remove some more Apache 1 left overs

- warning in configure.ac is relevant for the sapi/apache
- errors output redirected to /dev/null when checking Apache version

Merge branch 'PHP-7.4'

[ci skip]

Mention that zend_parse_parameters should not be tested

Also mention that --CREDITS-- section should not be included
anymore. We should also change qa.php.net to stop advertising
these.

[ci skip]

Disable jit on msan job

JIT code is not instrumented and as such incompatible with msan.
I'm not sure why this only started failing in the last build, it
shouldn't have worked before either...

Merge branch 'PHP-7.4'

* PHP-7.4:
  Move footer to the end of configure output

Move footer to the end of configure output

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove PTHREADS_ASSIGN_VARS

Remove PTHREADS_ASSIGN_VARS

This simplifies TSRM build steps a bit and avoids doing unnecessary
steps:
- The `PTHREADS_CHECK_COMPILE` can called inside the for loops only
  since this is only where the `$pthreads_checked` variable is used.
- Assigning variables can be then done only in the configure.ac
  once.
- use `m4_include()` instead of the `sinclude()` in the middle of
  the build steps.
- The `$threads_result` variable is not used in the code or in
  extensions.

Always generate interrupt check in jit

Even if zend_interrupt_function is NULL, we still need to perform
the interrupt check for timeouts (which do not use
zend_interrupt_function).

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove superfluous assignment

Remove superfluous assignment

Merge branch 'PHP-7.4'

* PHP-7.4:
  Eliminate useless checks

Eliminate useless checks