Kevin-Li-2025 opened a new pull request, #19899:
URL: https://github.com/apache/tvm/pull/19899
## What changed
This improves the diagnostics emitted when VM codegen sees a normal Relax
operator that was not lowered before codegen.
- Include actionable guidance in both VM and VMTIR codegen errors.
- Print the full offending Relax call, including attributes, instead of only
the operator name.
- Add VM build coverage using an unsupported `relax.nn.conv2d_transpose`
layout so the diagnostic exposes the layout attributes.
## Why
Issue #19694 reports that unsupported `conv{1,2,3}d_transpose` attributes
can pass through legalization and later fail with an opaque VM codegen error:
```text
CodeGenVM cannot handle this intrinsic now:
Op(relax.nn.conv2d_transpose)
```
Recent upstream work added dilation support, while the layout/out-layout
passthrough remains useful for external codegen flows. This PR keeps that
legalization behavior unchanged, but makes the eventual VM codegen failure
actionable by showing the exact unlowered call and attributes.
## Validation
- `git diff --check`
- `PYTHONDONTWRITEBYTECODE=1
/Users/yinxiaogou/.cache/codex-runtimes/codex-primary-runtime/dependencies/python/bin/python3
-m py_compile tests/python/relax/test_transform_legalize_ops_nn.py
tests/python/relax/test_vm_build.py`
Not run locally: the targeted pytest, because this checkout does not have a
built/importable TVM package available. CI should validate the new VM build
test.
Helps #19694.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
