yaxunl marked an inline comment as done.
yaxunl added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/FunctionImport.cpp:496
dbgs() << "ignored! No qualifying callee with summary found.\n");
continue;
}
----------------
tejohnson wrote:
> Probably better to issue an error here with the import failure reason?
My understanding is that the import failure reason is only available if
PrintImportFailures is enabled. Also it can only print the GUID and can not
print the failed callee name since it is not available, therefore the
information is cryptic. It seems to me the current error msg at line 1332 is
more suitable for common users. For compiler developers, they can enable
PrintImportFailures and see the reason of failed imports.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99683/new/
https://reviews.llvm.org/D99683
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
