Lunderberg commented on PR #14571: URL: https://github.com/apache/tvm/pull/14571#issuecomment-1503340908
Thank you for finding the error there. The rewrite were initially introduced to allow simplification of cases of `(x+const)%2` introduced by the `InjectSoftwarePipeline` pass, with equivalent changes made for `DetectIterMap` so that it could handle the simplified expressions. I agree that the `DetectIterMap` changes are definitely incorrect, and am going through the rewrite rules introduced in that PR to check whether they had a similar error, or whether the error was solely in the `IterMapSimplify` changes. -- 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]
