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: 003b42dc1f5398193c6f885d5a9559a0d976e79d / Junru Shao <[email protected]> refactor(abi): consolidate object subclass declarations Architecture: - Make details::IsObjectSubclass<T> the sole compile-time boundary for both complete Object inheritance detection and explicit forward-declaration trust. - Add TVM_FFI_DECLARE_OBJECT_SUBCLASS(Type) as the C++17 global-scope shortcut for specializing that trait. - Remove the parallel ABI-object marker hierarchy and ObjectPtr-specific container storage dispatch so ordinary TypeTraits<ObjectPtr<T>> and storage_enabled_v<T> share one invariant. - Remove type_traits.h dependencies and declarations that existed only to support the redundant marker path. Public Interfaces: - Generated ABI declarations now emit TVM_FFI_DECLARE_OBJECT_SUBCLASS instead of TVM_FFI_DECLARE_ABI_OBJECT. - The removed marker names were introduced only on this unmerged PR; no released API or ABI compatibility is affected. - Object layout and runtime ABI are unchanged. UI/UX: - none Behavioral Changes: - Declared incomplete Object subclasses remain valid in recursive and mutually recursive ObjectPtr container fields. - Undeclared incomplete types remain rejected, while complete C++ Object subclasses continue to be detected automatically. Docs: - No documentation update is needed because existing docs expose neither the removed unmerged implementation details nor the exact generated marker spelling. Tests: - Executed: uv run --no-sync pre-commit run --show-diff-on-failure --color=never --all-files - Result: passed all pre-commit hooks. - Executed: pinned clang-tidy 21.1.1 for tests/cpp/test_abi_object.cc and tests/cpp/test_object_ptr.cc - Result: passed both translation units. - Executed: uv run pytest for tests/python/test_dataclass_gen_abi_cpp.py - Result: 12 passed. - Executed: full Python unit suite - Result: 2391 passed, 18 skipped, 3 xfailed. - Executed: rebuilt tvm_ffi_tests and ran the direct GoogleTest binary; CTest also completed without failures - Result: all 450 enabled C++ tests passed; 2 tests remained disabled. Untested Edge Cases: - Rust tests were not executed because cargo and rustup are unavailable locally; this header/generator refactor does not modify Rust sources or bindings. Refs: https://github.com/apache/tvm-ffi/pull/670 Report URL: https://github.com/apache/tvm-ffi/actions/runs/29457056119 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
