Skip to content

vgg16 workload (1, 128, 122, 128, 3, 1, 1) error fixed and added to topi test#598

Merged
tqchen merged 1 commit into
apache:masterfrom
Laurawly:master
Oct 30, 2017
Merged

vgg16 workload (1, 128, 122, 128, 3, 1, 1) error fixed and added to topi test#598
tqchen merged 1 commit into
apache:masterfrom
Laurawly:master

Conversation

@Laurawly

Copy link
Copy Markdown
Contributor

No description provided.

@tqchen tqchen merged commit 3c89546 into apache:master Oct 30, 2017
@tqchen

tqchen commented Oct 30, 2017

Copy link
Copy Markdown
Member

cc @zhreshold

tlopex pushed a commit that referenced this pull request Jun 6, 2026
[Bump] tvm-ffi to 59da4c0

Bumps `3rdparty/tvm-ffi` from 98d0029 to 59da4c0, picking up seven
commits from apache/tvm-ffi:

- [FEAT] Optimize Expected<T> for minimal compiled code and efficiency
(#599)
- [FEAT] Streamline AccessPath/AccessStep print format (#598)
- [CI] Use uv tool run for sdist build instead of pipx (#600)
- [CORE] Make AnyView trivially copyable — match C-ABI struct passing
(#602)
- feat(python): add tensor methods to align with C++ APIs (#604)
- [FIX] Drop test_empty_tensor_attributes (numpy dlpack stride change)
(#607)
- [TEST] Relax test_shared_dag_hash_scaling_not_exponential ratio to 4x
(#612)

The bump range is additive for almost all of the above (performance
improvements to Expected<T>, new Python tensor methods, AnyView ABI
alignment, CI toolchain switch). One commit required a TVM-side
migration:

apache/tvm-ffi#598 moved the `__ffi_repr__` hooks for
`ffi::reflection::AccessPath` and `AccessStep` into tvm-ffi itself
(src/ffi/extra/reflection_extra.cc, compiled into libtvm_ffi.so).
TVM already registered the same hooks in src/ir/access_path_repr.cc,
causing a double-registration abort at library load time. This commit
removes the duplicate AccessPath/AccessStep `__ffi_repr__` registrations
from access_path_repr.cc, keeping only the `node.AsRepr` global function
that tvm-ffi does not provide. The format emitted by tvm-ffi's hooks is
equivalent for common cases; missing-item steps now use the
`[<missing:...>]` notation from tvm-ffi rather than the `...?` suffix
that TVM used previously.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants