[PATCH] D17212: [ThinLTO] Support for call graph in per-module and combined summary.

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 09:31:42 PST 2016


tejohnson added inline comments.

================
Comment at: include/llvm/IR/FunctionInfo.h:291
@@ +290,3 @@
+public:
+  FunctionSummaryIOHelper() {}
+
----------------
joker.eph wrote:
> Why this?
> If needed, replace with `FunctionSummaryIOHelper() = default;`
This was leftover from when I had a member variable that was being passed in and initialized in the constructor. Removing this now.


http://reviews.llvm.org/D17212





More information about the llvm-commits mailing list