<div dir="ltr">the reason for splitting the check is we don't need this check at all in llvm instrument,-- as it's done per function. but I guess it does not matter much to move it in to create -- it's only called once per function.<div><br></div><div><br><div><br></div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 20, 2016 at 3:04 PM, David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">davidxl added inline comments.<br>
<br>
================<br>
</span>Comment at: lib/CodeGen/CodeGenPGO.cpp:47<br>
@@ +46,3 @@<br>
<span class="">+  // Create PGOFuncName meta data.<br>
+  if (!llvm::getPGOFuncNameMetadata(*Fn))<br>
+    llvm::createPGOFuncNameMetadata(*Fn);<br>
</span>----------------<br>
This check be folded into the creator. The creator interface name also needs to be changed properly (created when needed)<br>
<br>
<br>
<a href="http://reviews.llvm.org/D18624" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18624</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>