[PATCH] D50881: [ThinLTO] Add option for printing import failure reasons
David Li via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 17 08:47:29 PDT 2018
davidxl added inline comments.
================
Comment at: include/llvm/Transforms/IPO/FunctionImport.h:44
+ None,
+ GlobalVar,
+ NotLive,
----------------
Add some comments on the reasons that are not obvious e.g, GlobalVar.
================
Comment at: include/llvm/Transforms/IPO/FunctionImport.h:56
+ ValueInfo VI;
+ CalleeInfo::HotnessType MaxHotness;
+ ImportFailureReason Reason;
----------------
Add a comment on MaxHotness.
Repository:
rL LLVM
https://reviews.llvm.org/D50881
More information about the llvm-commits
mailing list