Ref counting based object release is not applied if cycle is broken in destructor during GC run

Description

repro: https://3v4l.org/USrWu

see UPDATE below

It seems the GC can be improved. Currently, when a cycle is broken during GC run, a 2nd run of GC is needed to really release the object.

https://github.com/php/php-src/commit/b58d74547f "solves this issue" since PHP 8.2+, but is that the most efficient solution for everything? If a cycle (all cycles/references to an object) is explicitly broken during GC, is there anything againts releasing that object by standard refcounting?

Resulted in this output:

d
gc
bool(true) <- notice object is kept alive after the destructor has broken the cycle (and refcount dropped to 0)
gc rerun
bool(false)
bool(false)
done

But I expected this output instead:

d
gc
bool(false)
gc rerun
bool(false)
bool(false)
done

UPDATE:

Based on https://3v4l.org/mgoqd I would call this a bug as 2nd WeakReference::get() call is expected to always give the same result if no GC is run in between the calls.

PHP Version

any (tested PHP 7.4, 8.3)

Operating System

any

Bắt đầu bằng cách chạy bản tái hiện tại https://3v4l.org/USrWu và so sánh với ví dụ đã cập nhật tại https://3v4l.org/mgoqd, tập trung vào GC, destructor, số lượng tham chiếu và WeakReference::get(). Được xem là hoàn tất khi một chu kỳ bị phá vỡ trong một lần chạy GC giải phóng đối tượng ngay lập tức, để cả hai kiểm tra WeakReference đều khớp với đầu ra mong đợi mà không cần chạy GC lần thứ hai.