tqchen commented on PR #20350: URL: https://github.com/apache/tvm/pull/20350#issuecomment-5689062337
The latest verification found no blocking issues. Commit `17dbf5335` adds the intentional constant `AssertStmt::message_parts` skip comment and corrects the visitor test explanation to distinguish pointer `b` from `x`; assertions and executable code are unchanged. Both changed files are byte-identical to `be452a616f` after removing standalone line comments, and all applicable formatting/lint hooks pass. Runtime suites were not repeated for this comments-only change; the preceding build, 138 C++ tests, 139 focused Python tests, and 15 baseline-matched schedule diagnostics retain their original revision pins. Two deliberate consequences of unified traversal are retained: raw tile BufferVar arguments now reach use-site Var hooks (and therefore opaque-access consumers), and counting a TensorLoad source adds one to expression complexity for BufferVar-backed loads. These follow the requested unified Var/source traversal and removal of redundant TensorLoad overrides. BufferRegion tile arguments still dispatch through their precise consumer hooks. Coverage boundary: existing direct tile-call tests construct BufferRegions, and the script copy builder promotes whole buffers to regions. Existing CompactBufferRegion and PlanAndUpdateBufferAllocationLocation opaque-access tests use external calls; both files passed in the preceding focused run, but this does not establish direct raw-BufferVar tile-argument integration coverage. -- 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]
