<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 7, 2016 at 11:47 AM, Vedant Kumar <span dir="ltr"><<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vsk added inline comments.<br>
<span class=""><br>
================<br>
Comment at: lib/profile/InstrProfilingFile.c:149<br>
@@ +148,3 @@<br>
</span>+ * dumper. With profile merging enabled, each executable as well as any of<br>
+ * its instrumented shared libraries dump profile data into their own data file.<br>
+*/<br>
----------------<br>
Ok<br>
<br>
================<br>
Comment at: lib/profile/InstrProfilingFile.c:238<br>
@@ -126,2 +237,2 @@<br>
   lprofCurFilename.FilenamePat = FilenamePat;<br>
   /* Check the filename for "%p", which indicates a pid-substitution. */<br>
----------------<br>
I think "binary-%m.profraw" could be a common pattern; moreover I don't see a reason to not support this. Would removing this restriction require adding more code to getCurFilenameLength()/getCurFilename()?<br>
<span class=""><br></span></blockquote><div><br></div><div>Ok -- I can relax it.</div><div><br></div><div>David</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
================<br>
Comment at: test/profile/instrprof-basic.c:18<br>
@@ -5,2 +17,3 @@<br>
+// RUN: %clang_profuse=%t.m.profdata -o - -S -emit-llvm %s | FileCheck %s --check-prefix=COMMON --check-prefix=MERGE<br>
<br>
 int begin(int i) {<br>
----------------<br>
</span>Ok<br>
<br>
<br>
<a href="http://reviews.llvm.org/D21056" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21056</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>