You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/07/18 17:14:17 UTC

[GitHub] [tvm] quic-sanirudh commented on a diff in pull request #12082: [Pylint] Making hexagon tests pylint compliant Part 1 of N

quic-sanirudh commented on code in PR #12082:
URL: https://github.com/apache/tvm/pull/12082#discussion_r923625392


##########
tests/python/contrib/test_hexagon/test_benchmark_elemwise_add.py:
##########
@@ -124,6 +119,7 @@ def _get_irmod_elemwise_add(
         # Also: The VTCM budget is a very rough estimate, based only on experience.
         # Assuming that it's even reasonable to use a hard-coded estimate AT ALL, this number
         # may need tweaking.
+        # pylint: disable=unreachable

Review Comment:
   Yeah I wasn't sure if this needs to be available as the author seems to have left it intentionally unreachable, but I've commented it out for now and mentioned that it can be enabled when needed.



-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org