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/06/24 19:12:33 UTC
[GitHub] [tvm] kparzysz-quic opened a new pull request, #11888: [LLVM] Include LLVM headers in files that use them, not in llvm_common.h
kparzysz-quic opened a new pull request, #11888:
URL: https://github.com/apache/tvm/pull/11888
This is following the same principle we use everywhere else in TVM, that is, every source file includes headers that it depends on. While including unnecessary LLVM headers (which may happen by including llvm_common.h) is not actively harmful, it makes the header dependencies much less transparent.
--
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
[GitHub] [tvm] junrushao1994 merged pull request #11888: [LLVM] Include LLVM headers in files that use them, not in llvm_common.h
Posted by GitBox <gi...@apache.org>.
junrushao1994 merged PR #11888:
URL: https://github.com/apache/tvm/pull/11888
--
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