cbalint13 commented on PR #16425: URL: https://github.com/apache/tvm/pull/16425#issuecomment-2023388116
@Lunderberg , > We should not producing an error message when importing TVM. Can give me a script line how to reproduce (beside LLVM10 presence) ? Now that's sounds odd, if true I address a fix to this, will test all iterations llvm=range(10,19) > Thanks for the discussion @Lunderberg @cbalint13. I agree that we shouldn't remove the error message completely. Just thinking out loud - the problem here seems to be that the targets registered in tag.cc are parsed when loading tvm, is it possible to defer parsing of these registered targets to when they are actually used by the user? If turns true, we should take this one check out from ```LLVTargetInfo``` constructor and see a better place for it. I take care of this, if you don't mind this it will be a new PR. -- 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-unsubscr...@tvm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
