Lunderberg commented on issue #17230:
URL: https://github.com/apache/tvm/issues/17230#issuecomment-2263142421

   > `mod = mod.show()`
   
   This line looks like a bug in the test case.  Displaying a module is not a 
transform, and so `mod.show()` returns `None`.  This then correctly errors out 
by detecting the nullptr in the `IRModule`.
   
   (The error message may be improved by making `IRModule` be a non-nullable 
type, but a lot of the earlier portions of TVM rely on `ObjectRef` sub-types 
being nullable, so it would be a non-trivial change.  Would need to first find 
all instances `IRModule` that should be replaced by `Optional<IRModule>`.)


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