[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 @@
+  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.


More information about the llvm-commits mailing list