<div dir="ltr">yes please split it out.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 17, 2016 at 12:40 PM, Rong Xu <span dir="ltr"><<a href="mailto:xur@google.com" target="_blank">xur@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">xur added inline comments.<br>
<span class=""><br>
<br>
================<br>
Comment at: tools/llvm-profdata/llvm-<wbr>profdata.cpp:159<br>
</span><span class="">   for (auto &I : *Reader) {<br>
+    const StringRef FuncName = I.Name;<br>
     if (Error E = WC->Writer.addRecord(std::<wbr>move(I), Input.Weight)) {<br>
</span>----------------<br>
davidxl wrote:<br>
> unrelated change?<br>
this is for the use-after-move fix I mentioned in the description.<br>
You want to split to another patch?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D25687" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25687</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>