<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 19, 2016 at 4:51 PM, Vedant Kumar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vsk added a comment.<br>
<br>
Looks good! I just have one concern.<br>
<br>
<br>
================<br>
Comment at: lib/profile/InstrProfiling.h:155<br>
@@ +154,3 @@<br>
+ * not care about the default raw profile name. It is also useful to collect<br>
+ * more than more profile data files dumped in the same directory (Online<br>
+ * merge mode is turned on for instrumented programs with shared libs).<br>
----------------<br>
than -> and?<br>
<br>
================<br>
Comment at: lib/profile/InstrProfilingFile.c:435<br>
@@ +434,3 @@<br>
+<br>
+  if (lprofCurFilename.ProfilePathPrefix)<br>
+    return lprofCurFilename.ProfilePathPrefix;<br>
----------------<br>
Can this give the wrong answer if it comes after a call to __llvm_profile_set_filename()?<br></blockquote><div><br></div><div>Good catch. llvm_profile_set_filename should reset this. </div><div><br></div><div>David</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D22546" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22546</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>