Repository navigation
AOT: DatePeriod foreach SIGABRTs — snapshot still reads compileTimeLong after DateTime timestamp field split (re-#26937, ext/date/php_date.c) #33744
Description
Category
bug · AOT / DatePeriod · php-src-strict
Problem
Thin AOT foreach (new DatePeriod(...)) compiles then SIGABRTs (rc=134). Zend prints the dates. This is a regression of #26772 / #26937: JitDateTimeConstruct stamps compileTimeDateTimeTimestamp (#32691) so DateTime is not treated as an int, but JitDatePeriodConstruct still reads compileTimeLong, so the foreach snapshot is empty. VmIteratorForeach::compileReset then takes object-property foreach before the DatePeriod snapshot, and format() on a property value aborts.
./script/aot-smoke.sh is 8/8. Probed 2026-08-22.
| Repro | Zend 8.2 | AOT (master) |
|---|---|---|
foreach (new DatePeriod(new DateTimeImmutable('2020-01-01'), new DateInterval('P1D'), new DateTimeImmutable('2020-01-05')) as $d) echo $d->format('Y-m-d') |
2020-01-01,2020-01-02,2020-01-03,2020-01-04 |
rc=134 SIGABRT |
recurrences form new DatePeriod(..., 2) |
2020-01-01 2020-01-02 2020-01-03 |
rc=134 |
php-src reference
- php/php-src
ext/date/php_date.c—date_period_construct/ DatePeriod iterator - php/php-src
ext/date/php_date.stub.php—DatePeriodimplementsIteratorAggregate
PHP implementation target
ext/standard/JitDatePeriodConstruct.php— stamp snapshot fromcompileTimeDateTimeTimestamp/compileTimeTimezoneNamelib/VM/VmIteratorForeach.php— DatePeriod snapshot before object-property foreach (Regression: AOT DatePeriod foreach segfaults — VM/JIT match Zend (ext/date/php_date.c) #26772)lib/VM/VmObjectPropertyForeach.php— do not treat DatePeriod as instance-property foreach- No new
runtime/*.c
Repro
./script/docker-exec.sh -- bash -lc 'php bin/compile.php -o /tmp/dp test/repro/maintainer_gap_aot_dateperiod_foreach.php && /tmp/dp'Done when
- AOT foreach over compile-time DatePeriod (end-date + recurrences) matches Zend
-
./script/aot-smoke.shstays 8/8
Activity
PurHur commentedon Aug 22, 2026
claim: implementer-worker-lane-c — starting this run (lane overflow: #32122 and #31968 children closed; DatePeriod foreach AOT SIGABRT re-#26937)
AOT: DatePeriod foreach uses DateTime timestamp snapshots (#33744) (#…
PurHur commentedon Aug 22, 2026
merged: #33747 (
36aff1ea8b)Fix: DatePeriod foreach snapshot reads
compileTimeDateTimeTimestamp(notcompileTimeLong); snapshot RESET runs before object-property foreach; DateTime construct sync no longer clobbers other DateTimeImmutable locals.Verification
./script/docker-exec.sh -- bash -lc './script/aot-smoke.sh'→ 8/8./script/phpunit.sh --filter Issue33744DatePeriodForeachAotTest→ OK (5 tests)test/repro/issue_33744_dateperiod_foreach_aot.phpmatches Zend (end-date + recurrences + inline ctor)AOT: DatePeriod foreach uses DateTime timestamp snapshots (#33744) (#…
AOT: restore DateTime::diff after two locals via unnamed New_ stamp p…