Lunderberg commented on issue #17239: URL: https://github.com/apache/tvm/issues/17239#issuecomment-2269462421
I've submitted separate PR https://github.com/apache/tvm/pull/17242 which should provide a better error message (instead of a segfault) when this occurs. For (2), we may be able to improve it by checking `isinstance(args.struct_info, TupleStructInfo)` rather than `isinstance(args. relax.Tuple)`. This way, a tuple that was defined earlier in the function wouldn't be modified, and would produce an error message at an earlier point. The normalization (which would produce the required in-line Tuple) is suppressed during TVMScript parsing, since TVMScript is frequently used for writing test cases that violate Relax assumptions. Maybe we should tie the normalization to the existing `check_well_formed` flag, so those tests usually disable the well-formed checks as well. -- 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]
