d-smirnov commented on pull request #9842: URL: https://github.com/apache/tvm/pull/9842#issuecomment-1014351663
@tqchen Since we virtually have no NDEBUG macro available (due to LLVM issue), could a separate debug macro (i.e. TVM_DEBUG or similar) be considered instead to be present in pre-processor if `Debug` build configured via cmake? This is mostly because TVM_LOG_DEBUG needs specific steps for configuration which not as simple and straightforward as switching between Debug/Release builds. @junrushao1994 I might be wrong but it looks like Array does not need this type of checks. Could you advise on this, please -- 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]
