tqchen opened a new pull request, #19890:
URL: https://github.com/apache/tvm/pull/19890
This PR replaces Relax uses of void dtype sentinels for semantically absent
dtypes with explicit optional/null representations.
Summary:
- Make unknown `TensorType` dtype optional instead of represented as `void`.
- Convert optional Relax dtype attrs to optional values and update inference
paths to check presence directly.
- Represent `R.memory.view(..., dtype=None)` with `R.null_value()` at the
expression boundary, and lower to a concrete runtime dtype when required.
- Reject `R.memory.view(..., dtype=R.dtype("void"))` in inference instead of
treating the legacy void spelling as absence.
Validation:
- `/home/scratch.tianqic_sw/x86-ws/miniforge3/bin/cmake --build build
--parallel 16`
- `PYTHONPATH=$(pwd)/.local/python:$(pwd)/python
LD_LIBRARY_PATH=$(pwd)/build/lib:$(pwd)/build:${LD_LIBRARY_PATH:-}
/home/scratch.tianqic_sw/x86-ws/miniforge3/bin/python -m pytest
tests/python/relax/test_op_view.py -q -k 'not execute'`
- `/home/scratch.tianqic_sw/x86-ws/miniforge3/bin/python -m pre_commit run
--files src/relax/op/memory/view.cc tests/python/relax/test_op_view.py`
- Earlier branch validation covered the broader Relax
parser/printer/type-inference/create/nn/image/statistical/sorting/VM touched
areas and pre-commit over the full branch file set.
--
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]
