Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
👋 Hi! Thank you for contributing to the TileLang project. Please remember to run We appreciate you taking this step! Our team will review your contribution, and we look forward to your awesome work! 🚀 |
Summary
Refs #3006.
Normalize integer
is_lshift_requiredandis_rshift_requiredflags to nonzero boolean expressions in TileLang's existing operator-adapter layer. The documented integer flags otherwise reach aSelectcondition unchanged and abortq_multiply_shift_per_axislegalization with an internalis_bool()check.Existing boolean flags retain their IR unchanged. Add target-independent legalization/arithmetic regressions for constant and symbolic signed/unsigned flags, plus C-target CPU compilation and numerical execution regressions.
Scope
This fixes calls through the modern TileLang language facade and backend dialects that share it. Raw
tvm.tirxcalls and the legacy AST surface are unchanged; no TVM submodule or native C++ changes are included.Validation
format.sh --files, all applicable pre-commit hooks, pinned Ruff 0.16.6, andgit diff --checkpassedThe tests used an overlay containing the exact modified Python operator file over the wheel's unchanged native libraries, not a rebuilt checkout. The wheel's relevant
intrin_rule.ccis byte-identical to the current pinned TVM source. No GPU execution, NVCC compilation, LLVM runtime, or full-suite validation is claimed. A neighboring CUDA-marked test file could not collect because existing GPU feature probing aborts without a CUDA driver.Provenance
The implementation and tests are AI-generated and were independently checked in a separate AI review pass. This draft is available for maintainer review.