The GitHub Actions job "CI" on tvm-ffi.git/junrushao/2026-07-15/generate-layout-compatible-cpp-object-declarations has failed. Run started by GitHub user junrushao (triggered by junrushao).
Head commit for run: 2a8281041874a80b5f1b6b783997d6b75e19781a / Junru Shao <[email protected]> refactor(abi): restore object pointer behavior Architecture: - Isolate generated incomplete-object support to TVM_FFI_DECLARE_OBJECT_SUBCLASS and restore all existing ObjectPtr, WeakObjectPtr, hashing, and subsumption implementation paths from upstream/main. - Keep the generated forward-declaration specialization orthogonal to core pointer ownership and conversion semantics. Public Interfaces: - Retain TVM_FFI_DECLARE_OBJECT_SUBCLASS as the only object.h addition relative to upstream/main. - Remove the unmerged conversion-constraint changes, so existing ObjectPtr and WeakObjectPtr source compatibility and behavior remain unchanged. - No C ABI, layout, Python API, or generated field type changes are introduced. UI/UX: - none Behavioral Changes: - Restore upstream constructor validation, bool conversion, ObjectPtrHash access, and type_subsumes_v behavior. - Remove negative is_constructible assertions that depended on the reverted SFINAE implementation rather than the generated ABI declaration feature. - Recursive and mutually recursive generated ObjectPtr fields remain enabled through the retained subclass-declaration macro. Docs: - No documentation update is needed because this cleanup removes unrelated unmerged behavior; the retained macro is documented inline and generated-output tests cover its contract. Tests: - Executed: full CTest suite - Result: all 451 enabled tests passed; 2 remained disabled. - Executed: full Python unit suite - Result: 2393 passed, 18 skipped, 3 xfailed. - Executed: pre-commit on all files - Result: passed all hooks. - Executed: pinned clang-tidy 21.1.1 on tests/cpp/test_abi_object.cc and tests/cpp/test_object_ptr.cc - Result: passed both translation units. - Executed: git diff --check - Result: passed. Untested Edge Cases: - Rust tests were not executed because cargo and rustup are unavailable locally; this cleanup changes no Rust source or binding. - Linux and Windows compiler matrices remain CI-only; the full local C++ suite and targeted clang-tidy cover the restored pointer paths. Refs: https://github.com/apache/tvm-ffi/pull/670 Report URL: https://github.com/apache/tvm-ffi/actions/runs/29458141073 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
