Commits on Feb 23, 2024

1. [Transform] Check for zero-param operators in LiftTransformParams …

   Prior to this commit, `LiftTransformParams` would extract out all
   variable binding that have no runtime dependencies.  As a result,
   expressions such as `R.zeros([16], "int32")` would be extracted out
   into the parameter transformation, even though they do not depend on
   any parameters.
   
   This commit updates `LiftTransformParams` to only output variables
   that depend on at least one compile-time parameter.
   
   The unit test for this functionality also found that `relax::Call` was
   erroneously calling `MarkGraphNode` in `SEqualReduce` and
   `SHashReduce`.  This should only be called for nodes that have have
   reference equality, such as `relax::Var`, and not for composite
   objects.  This caused erroneous failures in the unit test when two
   instances of `R.zeros([16], "int32")` were being compared by reference
   equality in `StructuralEqual`.  These extra calls to `MarkGraphNode`
   have been removed.

   

   Lunderberg committed Feb 23, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 2390033
   • Browse repository at this point

   Copy the full SHA

   2390033 View commit details

   Browse the repository at this point in the history