youxiudeshouyeren commented on PR #15682: URL: https://github.com/apache/tvm/pull/15682#issuecomment-1711006449
> Thanks for the PR! @youxiudeshouyeren. It seems that this example should have been optimized by `SimplifyClip`/`SimplifyClipAndConsecutiveCast` passes, but it was not executed. Perhaps you need to prioritize checking why existing passes have not been optimized to this pattern. They are different optimizations and both have different patterns. SimplifyClipAndConsecutiveCast targets clip(int32)-cast(uint8)->cast(int32). This simplification targets clip(int32)->cast(int8)->clip(int8). I'm sure the pass SimplifyClipAndConsecutiveCast is enabled, but it doesn't eliminate this redundancy in yolov5. -- 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]
