[PATCH] Value profiling - patchset 2 - merge intended

David Blaikie dblaikie at gmail.com
Mon Jun 22 14:05:15 PDT 2015


================
Comment at: include/llvm/ProfileData/InstrProf.h:50-51
@@ +49,4 @@
+  InstrProfRecord() {}
+  InstrProfRecord(StringRef Name, uint64_t Hash, std::vector<uint64_t> Counts)
+      : Name(Name), Hash(Hash), Counts(std::move(Counts)) {}
+  StringRef Name;
----------------
Yep, this looks good to me now - thanks!

I'll leave it to Justin to review the substance of the patch - I don't have any knowledge here.

http://reviews.llvm.org/D10493

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list