quic-sanirudh opened a new pull request, #16795:
URL: https://github.com/apache/tvm/pull/16795

   This patch makes a small change to compare the values of IntImmNode to see 
if they're equal when performing a deep_equal of expressions. This is to try 
and align it with how the 
[`PEqualChecker<IntImm>`](https://github.com/apache/tvm/blob/b2204ae6988c7745ea9736340ccd900bc21ae821/src/arith/pattern_match.h#L168)
 works where we only compare the values if both are IntImm.
   
   This caused some simplifications to be inconsistent based on whether we used 
IntImmNode or PrimExpr to pass an integer between different passes, and it 
seemed to make more sense to say that if the values are equal, then we can 
conclude the immediates are equal.


-- 
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]

Reply via email to