[PATCH] D25687: [PGO] fix bogus warning for merging empty llvm profile file

Rong Xu via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 17 12:40:21 PDT 2016


xur added inline comments.


================
Comment at: tools/llvm-profdata/llvm-profdata.cpp:159
   for (auto &I : *Reader) {
+    const StringRef FuncName = I.Name;
     if (Error E = WC->Writer.addRecord(std::move(I), Input.Weight)) {
----------------
davidxl wrote:
> unrelated change?
this is for the use-after-move fix I mentioned in the description. 
You want to split to another patch?


https://reviews.llvm.org/D25687





More information about the llvm-commits mailing list