[PATCH] D19397: Initial patch for inlining report

[Quentin Colombet via llvm-commits]

qcolombet requested changes to this revision.
qcolombet added a reviewer: qcolombet.
This revision now requires changes to proceed.

================
Comment at: include/llvm/Analysis/CallGraphReport.h:26
@@ +25,3 @@
+// on the call graph. 
+class CallGraphReport { 
+public: 
----------------
I believe we should have something of higher level like a Report class.
Other type of optimizations may report some stuff as well.
I was also wondering how different this is from the Remark mechanism and assuming we indeed need a new mechanism for reports, I think we should do something similar to the remarks.


http://reviews.llvm.org/D19397