The GitHub Actions job "CI" on tvm-ffi.git/pyobject has failed.
Run started by GitHub user cyx-6 (triggered by cyx-6).

Head commit for run:
1a0e7a73dd3d13a530efe7846816f34a64233621 / Yaxing Cai <[email protected]>
[REFACTOR][Python] Address review on pyobject-tying helpers

Apply Tianqi's PR #593 review feedback to the pyobject-tying layer:

- Drop the kPyHeaderOffset constant; use sizeof(PyCustomAllocHeader)
  directly at every site (and in the static_asserts).
- Rename TVMFFIPyTlsReviveSlot -> TVMFFIPyTLSReviveSlot and make its sole
  access primitive a swap-exchange (store next, return prior). The slot is
  typed PyObject*, and taking the block is TVMFFIPyTLSReviveSlot(nullptr) --
  read-and-clear in one step, no separate clear to forget.
- Document \param/\return on TVMFFIPyTryGetAttachedPyObject.
- Reorder the slot functions into execution order (TpAlloc -> TpDealloc ->
  TpFree) with the DEALLOC_TRANSIT marker declared between them.
- Replace the hand-rolled TVMFFIPyCObjectHead layout mirror with a Cython
  cdef public accessor TVMFFICyObjectGetCHandlePtr, so the chandle field
  offset comes from the real cdef class layout instead of a hardcoded one.

Build clean; full Python suite passes (2336 passed, 18 skipped, 2 xfailed).

Report URL: https://github.com/apache/tvm-ffi/actions/runs/26958827885

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to