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: bb6bd124acaf4fc2808055c46d02cb95f8119464 / Junru Shao <[email protected]> fix(dataclasses): generate raw object pointers in containers Architecture: - Model object-valued container arguments as raw TObj* carriers so generated declarations can reference mutually recursive and otherwise incomplete ABI object classes without relying on rejected container shorthand. - Preserve ownership for standalone object fields with ObjectPtr<TObj>, and admit incomplete raw pointer storage only for unqualified types marked by TVM_FFI_DECLARE_ABI_OBJECT. - Route raw pointer TypeTraits selection through the incomplete-safe object-subclass trait; undeclared incomplete types still fail storage eligibility. Public Interfaces: - Change the C++ text emitted by gen_abi_cpp for object elements in Array, List, Map, and Dict from ObjectPtr<TObj> to TObj*. - Keep standalone object fields and built-in public wrappers unchanged; pointer size, alignment, C ABI, schemas, configuration, and CLI surfaces are unchanged. UI/UX: - none Behavioral Changes: - Generated native, recursive, and mutually recursive object-container declarations now compile without instantiating invalid shorthand or std::is_base_of on incomplete classes. - Raw pointer container storage remains disabled for undeclared incomplete types and cv-qualified pointees. Docs: - No docs updated; existing guidance specifies layout-compatible declaration generation rather than an exact carrier spelling, and executable expected-output plus native compile tests define the representation contract. Tests: - Executed: pre-commit run --all-files - Executed: uv run --no-project --with clang-tidy==21.1.1 python tests/lint/clang_tidy_precommit.py --build-dir=build-pre-commit --jobs=2 tests/cpp/test_abi_object.cc tests/cpp/test_object_ptr.cc - Executed: uv run pytest -q tests/python/test_dataclass_gen_abi_cpp.py - Executed: uv run pytest -q tests/python - Executed: cmake --build build-pre-commit --parallel 2 --config Debug --target tvm_ffi_tests - Executed: ctest -C Debug --test-dir build-pre-commit --output-on-failure - Result: all executed checks passed; the generator suite passed 12 tests, Python passed 2338 with 18 skipped and 3 xfailed, and CTest passed all 450 enabled cases with 2 disabled. - Not executed: cargo test because Cargo and Rustup are unavailable in the local environment. Untested Edge Cases: - Rust workspace tests were unavailable locally; risk is low because this change is confined to C++ traits and Python declaration generation. - Linux and Windows compiler matrices remain CI-only; native generated-code compilation, full C++ tests, and pinned clang-tidy provide local template-instantiation coverage. Refs: https://github.com/apache/tvm-ffi/pull/670 Report URL: https://github.com/apache/tvm-ffi/actions/runs/29453515253 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
