[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